)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6598,"name":"Berezovsky Irena","email":"irenab.dev@gmail.com","username":"irenab"},"change_message_id":"8262c5deb8ebf179d192a4703ab1e3f80665e651","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The changed code resolved method names by constructing method name"},{"line_number":10,"context_line":"string from parameters. Since the code handles just 6 different handlers"},{"line_number":11,"context_line":"across 2 scenarios, it doesn\u0027t make a lot of sense to that."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1737339"},{"line_number":14,"context_line":"Change-Id: Icbe50c0c74bcfaa89903e18808c3beca616638f8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"df87a7cf_d8a91190","line":12,"updated":"2017-12-10 06:32:46.000000000","message":"Commit message is not clear. It does not say clearly what the commit is about","commit_id":"579677bc08df06d32367ff7b77b9e7f42587eddd"},{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"17f7ac4cb6fcbee8e18c72b6859cbd03fcebd8b1","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The changed code resolved method names by constructing method name"},{"line_number":10,"context_line":"string from parameters. Since the code handles just 6 different handlers"},{"line_number":11,"context_line":"across 2 scenarios, it doesn\u0027t make a lot of sense to that."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Closes-Bug: #1737339"},{"line_number":14,"context_line":"Change-Id: Icbe50c0c74bcfaa89903e18808c3beca616638f8"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"df87a7cf_dedb394c","line":12,"in_reply_to":"df87a7cf_d8a91190","updated":"2017-12-10 12:03:05.000000000","message":"Will reword","commit_id":"579677bc08df06d32367ff7b77b9e7f42587eddd"}],"dragonflow/controller/topology.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"e0989e0c88ab69361e3ba72e29b3a04a4951bd32","unresolved":false,"context_lines":[{"line_number":69,"context_line":"            return"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"        try:"},{"line_number":72,"context_line":"            if orig_ovs_port is None:"},{"line_number":73,"context_line":"                self._handle_ovs_port_added(ovs_port)"},{"line_number":74,"context_line":"            else:"},{"line_number":75,"context_line":"                self._handle_ovs_port_updated(ovs_port)"},{"line_number":76,"context_line":"        except Exception:"},{"line_number":77,"context_line":"            LOG.exception("},{"line_number":78,"context_line":"                \"Exception occurred when handling port online event\")"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f485f77_3b1759f0","line":75,"range":{"start_line":72,"start_character":0,"end_line":75,"end_character":55},"updated":"2017-11-15 04:53:44.000000000","message":"You can go a step further and split ovs_port_updated and ovs_port_created.\n\nI think we can also do away with the exception handling - doesn\u0027t the event emitting code do that for us?","commit_id":"925e82da819b955ff39d1b21f39b8ae8ba5e7f59"}]}