)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"74d2eb9aea3afd52aeb8b50007dd1da21465b44a","unresolved":false,"context_lines":[{"line_number":9,"context_line":"six.reraise can be converted into \"raise new_exc from original_exc\""},{"line_number":10,"context_line":"introduced in python3."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"In case of horizon.exceptions.handle(), six.reraise was used to"},{"line_number":13,"context_line":"raise the original exception again, so we can convert it into \"raise\"."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"six is removed from requirements.txt."},{"line_number":16,"context_line":"Note that we need to keep six in lower-constraints.txt"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_ea1b261a","line":13,"range":{"start_line":12,"start_character":40,"end_line":13,"end_character":69},"updated":"2020-01-17 16:18:51.000000000","message":"Great, I didn\u0027t check that we don\u0027t change the traceback when I did my patches!","commit_id":"4fc1b9c4249a788fb1d59510096f72033eb85328"}],"horizon/exceptions.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"57df9faecc91499797365a77d7cdcbfb3bb05f82","unresolved":false,"context_lines":[{"line_number":345,"context_line":"        if ret:"},{"line_number":346,"context_line":"            return ret"},{"line_number":347,"context_line":""},{"line_number":348,"context_line":"    six.reraise(exc_type, exc_value, exc_traceback)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3fa7e38b_27e422a8","side":"PARENT","line":348,"updated":"2020-01-16 22:38:48.000000000","message":"(Note for reviewers)\nThis might need a detail explanation.\nIn the original code, exc_* variables are referred. They are assigned at two places: L.294 from sys.exc_info() and L.307 if handled\u003dTrue.\nIf handled\u003dTrue, wrap is set to True and if wrap is True the logic goes to L.336 and it never reaches here.\nThus, exc_* variables used here are always from L.294 sys.exc_info().\nRaising an exception using a return value from sys.exc_info() is just to re-raise an original exception, so we can convert it into \"raise\".","commit_id":"d870b40583b2b735e43a061110380717929598ed"}]}