)]}'
{"tests/unit/db/sqlalchemy/test_sqlalchemy.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"d510df7f455bf497cc6279e45ef8e023ca7b7b04","unresolved":false,"context_lines":[{"line_number":422,"context_line":"        def execute(self, sql):"},{"line_number":423,"context_line":"            if not self._connected:"},{"line_number":424,"context_line":"                self.connect()"},{"line_number":425,"context_line":"                self._connected \u003d True"},{"line_number":426,"context_line":"            self._execs.append(sql)"},{"line_number":427,"context_line":"            return MysqlSetCallbackTest.FakeResultSet(self._realmode)"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2F8Rk%3D","line":425,"updated":"2014-03-12 19:37:50.000000000","message":"Could have left the\n\n cb \u003d self._cbs.get(\u0027checkout\u0027, lambda *x, **y: None)\n cb(dbapi_con, connection_rec, connection_proxy)\n\n(this class would have to store the connection rather than have self._connected \u003d True/False.)\n\nto verify that the checkout handler isn\u0027t there anymore and in case a checkout handler is added in future it\u0027ll be more obvious how to handle it.","commit_id":"d1988b96266cf275fdb2c2cee01adc46797004fb"},{"author":{"_account_id":6928,"name":"Ben Nemec","email":"openstack@nemebean.com","username":"bnemec"},"change_message_id":"51d734f75cd1527b4246216b925f3cfe57b02b63","unresolved":false,"context_lines":[{"line_number":422,"context_line":"        def execute(self, sql):"},{"line_number":423,"context_line":"            if not self._connected:"},{"line_number":424,"context_line":"                self.connect()"},{"line_number":425,"context_line":"                self._connected \u003d True"},{"line_number":426,"context_line":"            self._execs.append(sql)"},{"line_number":427,"context_line":"            return MysqlSetCallbackTest.FakeResultSet(self._realmode)"},{"line_number":428,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAWH%2F%2F0NU%3D","line":425,"in_reply_to":"AAAAWH%2F%2F8Rk%3D","updated":"2014-03-13 16:23:44.000000000","message":"Good point.  If I end up pushing another patch set I\u0027ll look into it.","commit_id":"d1988b96266cf275fdb2c2cee01adc46797004fb"}]}