)]}'
{"oslo/db/sqlalchemy/exc_filters.py":[{"author":{"_account_id":12363,"name":"Oleksii Chuprykov","email":"a.chuprikoff@gmail.com","username":"ochuprykov"},"change_message_id":"9d3c03a8ec14513fb30536acbd73c53bfcce4d51","unresolved":false,"context_lines":[{"line_number":309,"context_line":"                if super_ in per_dialect:"},{"line_number":310,"context_line":"                    regexp_reg \u003d per_dialect[super_]"},{"line_number":311,"context_line":"                    for fn, regexp in regexp_reg:"},{"line_number":312,"context_line":"                        match \u003d regexp.match(exc.args[0])"},{"line_number":313,"context_line":"                        if match:"},{"line_number":314,"context_line":"                            try:"},{"line_number":315,"context_line":"                                fn("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_a2252dc0","line":312,"updated":"2014-08-04 13:17:43.000000000","message":"Can we use str(exc) instead of exc.args[0]?","commit_id":"584a8834889f7f5fac4b598e63bbcc8e1dd53abe"},{"author":{"_account_id":7763,"name":"Alexei Kornienko","email":"akornienko@mirantis.com","username":"akornienko"},"change_message_id":"cc3099cbb56e1bd738bc91d1dfa3af8a4ccdf84e","unresolved":false,"context_lines":[{"line_number":309,"context_line":"                if super_ in per_dialect:"},{"line_number":310,"context_line":"                    regexp_reg \u003d per_dialect[super_]"},{"line_number":311,"context_line":"                    for fn, regexp in regexp_reg:"},{"line_number":312,"context_line":"                        match \u003d regexp.match(exc.args[0])"},{"line_number":313,"context_line":"                        if match:"},{"line_number":314,"context_line":"                            try:"},{"line_number":315,"context_line":"                                fn("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_5da992db","line":312,"in_reply_to":"baada198_a2252dc0","updated":"2014-08-04 13:27:22.000000000","message":"We could use it but we would have to change regexp to do it. Cause of this I\u0027ve used this arg","commit_id":"584a8834889f7f5fac4b598e63bbcc8e1dd53abe"},{"author":{"_account_id":7763,"name":"Alexei Kornienko","email":"akornienko@mirantis.com","username":"akornienko"},"change_message_id":"ba12fa4d5ba525b853a8eddbec50d182382f0467","unresolved":false,"context_lines":[{"line_number":309,"context_line":"                if super_ in per_dialect:"},{"line_number":310,"context_line":"                    regexp_reg \u003d per_dialect[super_]"},{"line_number":311,"context_line":"                    for fn, regexp in regexp_reg:"},{"line_number":312,"context_line":"                        match \u003d regexp.match(exc.args[0])"},{"line_number":313,"context_line":"                        if match:"},{"line_number":314,"context_line":"                            try:"},{"line_number":315,"context_line":"                                fn("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_5dfc323f","line":312,"in_reply_to":"baada198_a2252dc0","updated":"2014-08-05 12:14:17.000000000","message":"We could use it but we would have to change regexp to do it. Cause of this I\u0027ve used this arg","commit_id":"584a8834889f7f5fac4b598e63bbcc8e1dd53abe"}],"tests/sqlalchemy/test_exc_filters.py":[{"author":{"_account_id":12363,"name":"Oleksii Chuprykov","email":"a.chuprikoff@gmail.com","username":"ochuprykov"},"change_message_id":"9d3c03a8ec14513fb30536acbd73c53bfcce4d51","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        self.assertEqual("},{"line_number":118,"context_line":"            \"(ProgrammingError) Error 123, you made a \""},{"line_number":119,"context_line":"            \"mistake \u0027select you_made_a_programming_error\u0027 ()\","},{"line_number":120,"context_line":"            matched.args[0])"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    def test_generic_dbapi_disconnect(self):"},{"line_number":123,"context_line":"        matched \u003d self._run_test("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_425d3147","line":120,"updated":"2014-08-04 13:17:43.000000000","message":"Can we use str(matched) for more elegance?","commit_id":"584a8834889f7f5fac4b598e63bbcc8e1dd53abe"},{"author":{"_account_id":7763,"name":"Alexei Kornienko","email":"akornienko@mirantis.com","username":"akornienko"},"change_message_id":"9206b0e54f801504fea9f2f397791cda704b2712","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        self.assertEqual("},{"line_number":118,"context_line":"            \"(ProgrammingError) Error 123, you made a \""},{"line_number":119,"context_line":"            \"mistake \u0027select you_made_a_programming_error\u0027 ()\","},{"line_number":120,"context_line":"            matched.args[0])"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    def test_generic_dbapi_disconnect(self):"},{"line_number":123,"context_line":"        matched \u003d self._run_test("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_0e8ed4d1","line":120,"in_reply_to":"baada198_2e6478b2","updated":"2014-08-05 15:05:56.000000000","message":"Could you please check the same comment in the other file? I cannot use str(matched) or whatever(matched) since it won\u0027t work with existing regex.","commit_id":"584a8834889f7f5fac4b598e63bbcc8e1dd53abe"},{"author":{"_account_id":7536,"name":"Ilya Pekelny","email":"ipekelny@mirantis.com","username":"I159"},"change_message_id":"3143f4f614c65f071b5637a576dbb7405522f245","unresolved":false,"context_lines":[{"line_number":117,"context_line":"        self.assertEqual("},{"line_number":118,"context_line":"            \"(ProgrammingError) Error 123, you made a \""},{"line_number":119,"context_line":"            \"mistake \u0027select you_made_a_programming_error\u0027 ()\","},{"line_number":120,"context_line":"            matched.args[0])"},{"line_number":121,"context_line":""},{"line_number":122,"context_line":"    def test_generic_dbapi_disconnect(self):"},{"line_number":123,"context_line":"        matched \u003d self._run_test("}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_2e6478b2","line":120,"in_reply_to":"baada198_425d3147","updated":"2014-08-05 15:02:29.000000000","message":"six.text_type, not str, str is incompatible. \nnit: Yes, wrapped into text_type, looks better.","commit_id":"584a8834889f7f5fac4b598e63bbcc8e1dd53abe"}],"tests/sqlalchemy/test_handle_error.py":[{"author":{"_account_id":12363,"name":"Oleksii Chuprykov","email":"a.chuprikoff@gmail.com","username":"ochuprykov"},"change_message_id":"9d3c03a8ec14513fb30536acbd73c53bfcce4d51","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                self.engine.execute, \"SELECT \u0027ERROR ONE\u0027 FROM I_DONT_EXIST\""},{"line_number":63,"context_line":"            )"},{"line_number":64,"context_line":"            self.assertEqual(1, patched.call_count)"},{"line_number":65,"context_line":"            self.assertEqual(\"my exception\", matchee.args[0])"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"    def test_exception_event_non_altered(self):"},{"line_number":68,"context_line":"        self._fixture()"}],"source_content_type":"text/x-python","patch_set":2,"id":"baada198_5d0d1234","line":65,"updated":"2014-08-04 13:17:43.000000000","message":"can we use str(matchee)?","commit_id":"584a8834889f7f5fac4b598e63bbcc8e1dd53abe"}]}