)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"d87246bd7b8a4ea6ea405a3ad8e3443bb9e176b2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     gengchc2 \u003cgeng.changcai2@zte.com.cn\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-09-12 02:41:37 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"dd __ne__ built-in function for dragonflow"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In Python 3 __ne__ by default delegates to __eq__ and inverts the"},{"line_number":10,"context_line":"result,but in Python 2 they urge you to define __ne__ when you define"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"9a89bdaa_37ca3150","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":2},"updated":"2016-09-12 02:54:30.000000000","message":"add?","commit_id":"f843bd954c8309d51dbfe05b7b63f9e72434ef38"}],"dragonflow/db/api_nb.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"57c167a9991f54a6d6534638dc8a2e5179f2f1b6","unresolved":false,"context_lines":[{"line_number":820,"context_line":"        else:"},{"line_number":821,"context_line":"            return False"},{"line_number":822,"context_line":"    def __ne__(self, other):"},{"line_number":823,"context_line":"        return not self.__eq__(other)"},{"line_number":824,"context_line":""},{"line_number":825,"context_line":""},{"line_number":826,"context_line":"class Subnet(DbStoreObject):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_a713a95b","line":823,"updated":"2016-09-05 08:59:56.000000000","message":"Please do not use __eq__. I think \u0027not (self \u003d\u003d other)\u0027 is better.","commit_id":"193eca5f6ebb6bc61804df99ad43d55302805c03"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"57c167a9991f54a6d6534638dc8a2e5179f2f1b6","unresolved":false,"context_lines":[{"line_number":991,"context_line":"        return self.get_id() \u003d\u003d other.get_id()"},{"line_number":992,"context_line":""},{"line_number":993,"context_line":"    def __ne__(self, other):"},{"line_number":994,"context_line":"        return not self.__eq__(other)"},{"line_number":995,"context_line":""},{"line_number":996,"context_line":"    def __str__(self):"},{"line_number":997,"context_line":"        return self.router_port.__str__()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_e71db167","line":994,"updated":"2016-09-05 08:59:56.000000000","message":"Same here.","commit_id":"193eca5f6ebb6bc61804df99ad43d55302805c03"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"57c167a9991f54a6d6534638dc8a2e5179f2f1b6","unresolved":false,"context_lines":[{"line_number":1064,"context_line":"        return self.get_id() \u003d\u003d other.get_id()"},{"line_number":1065,"context_line":""},{"line_number":1066,"context_line":"    def __ne__(self, other):"},{"line_number":1067,"context_line":"        return not self.__eq__(other)"},{"line_number":1068,"context_line":""},{"line_number":1069,"context_line":"    def __str__(self):"},{"line_number":1070,"context_line":"        return self.secrule.__str__()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_2726b9ba","line":1067,"updated":"2016-09-05 08:59:56.000000000","message":"And here.","commit_id":"193eca5f6ebb6bc61804df99ad43d55302805c03"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"2ba079ba5df8e1d2e505ed06a25ed4d0aeff9bbd","unresolved":false,"context_lines":[{"line_number":818,"context_line":"        if isinstance(other, self.__class__):"},{"line_number":819,"context_line":"            return self.lswitch \u003d\u003d other.lswitch"},{"line_number":820,"context_line":"        else:"},{"line_number":821,"context_line":"            return False"},{"line_number":822,"context_line":"    def __ne__(self, other):"},{"line_number":823,"context_line":"        return not (self \u003d\u003d other)"},{"line_number":824,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_6d46377c","line":821,"updated":"2016-09-08 08:21:27.000000000","message":"A new line here is required for pep8","commit_id":"f908fb280fb8ee6f1d22accea39988b7a0b78162"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"2ba079ba5df8e1d2e505ed06a25ed4d0aeff9bbd","unresolved":false,"context_lines":[{"line_number":991,"context_line":"        return self.get_id() \u003d\u003d other.get_id()"},{"line_number":992,"context_line":""},{"line_number":993,"context_line":"    def __ne__(self, other):"},{"line_number":994,"context_line":"        return not self.__eq__(other)"},{"line_number":995,"context_line":""},{"line_number":996,"context_line":"    def __str__(self):"},{"line_number":997,"context_line":"        return self.router_port.__str__()"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_8dfc5bb9","line":994,"range":{"start_line":994,"start_character":0,"end_line":994,"end_character":37},"updated":"2016-09-08 08:21:27.000000000","message":"Why not use not (self \u003d\u003d other)","commit_id":"f908fb280fb8ee6f1d22accea39988b7a0b78162"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"d87246bd7b8a4ea6ea405a3ad8e3443bb9e176b2","unresolved":false,"context_lines":[{"line_number":992,"context_line":"        return self.get_id() \u003d\u003d other.get_id()"},{"line_number":993,"context_line":""},{"line_number":994,"context_line":"    def __ne__(self, other):"},{"line_number":995,"context_line":"        return not self.__eq__(other)"},{"line_number":996,"context_line":""},{"line_number":997,"context_line":"    def __str__(self):"},{"line_number":998,"context_line":"        return self.router_port.__str__()"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_57c5f53a","line":995,"range":{"start_line":995,"start_character":0,"end_line":995,"end_character":37},"updated":"2016-09-12 02:54:30.000000000","message":"use not (self \u003d\u003d other) to keep consistent with other changes in this patch.","commit_id":"f843bd954c8309d51dbfe05b7b63f9e72434ef38"},{"author":{"_account_id":23235,"name":"Lihi Wishnitzer","email":"lihiwish@gmail.com","username":"lihiwish"},"change_message_id":"0b7897942a69b4d199fbfba3876cc319b0d173e3","unresolved":false,"context_lines":[{"line_number":1065,"context_line":"        return self.get_id() \u003d\u003d other.get_id()"},{"line_number":1066,"context_line":""},{"line_number":1067,"context_line":"    def __ne__(self, other):"},{"line_number":1068,"context_line":"        return not self.__eq__(other)"},{"line_number":1069,"context_line":""},{"line_number":1070,"context_line":"    def __str__(self):"},{"line_number":1071,"context_line":"        return self.secrule.__str__()"}],"source_content_type":"text/x-python","patch_set":4,"id":"9a89bdaa_74dd8041","line":1068,"range":{"start_line":1068,"start_character":19,"end_line":1068,"end_character":37},"updated":"2016-09-12 11:52:41.000000000","message":"Same comment as the others. It is better to use (self \u003d\u003d other)","commit_id":"f843bd954c8309d51dbfe05b7b63f9e72434ef38"}],"dragonflow/db/drivers/ovsdb_nb_impl.py":[{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"57c167a9991f54a6d6534638dc8a2e5179f2f1b6","unresolved":false,"context_lines":[{"line_number":230,"context_line":"    def __eq__(self, other):"},{"line_number":231,"context_line":"        return self.name \u003d\u003d other.get_name()"},{"line_number":232,"context_line":"    def __ne__(self, other):"},{"line_number":233,"context_line":"        return not self.__eq__(other)"},{"line_number":234,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_e734d1df","line":233,"updated":"2016-09-05 08:59:56.000000000","message":"And here (re: use of __eq__)","commit_id":"193eca5f6ebb6bc61804df99ad43d55302805c03"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"2ba079ba5df8e1d2e505ed06a25ed4d0aeff9bbd","unresolved":false,"context_lines":[{"line_number":228,"context_line":"        return self.network"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def __eq__(self, other):"},{"line_number":231,"context_line":"        return self.name \u003d\u003d other.get_name()"},{"line_number":232,"context_line":"    def __ne__(self, other):"},{"line_number":233,"context_line":"        return not (self \u003d\u003d other)"},{"line_number":234,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_0d34cbf9","line":231,"updated":"2016-09-08 08:21:27.000000000","message":"A new line here is required for pep8","commit_id":"f908fb280fb8ee6f1d22accea39988b7a0b78162"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"2ba079ba5df8e1d2e505ed06a25ed4d0aeff9bbd","unresolved":false,"context_lines":[{"line_number":231,"context_line":"        return self.name \u003d\u003d other.get_name()"},{"line_number":232,"context_line":"    def __ne__(self, other):"},{"line_number":233,"context_line":"        return not (self \u003d\u003d other)"},{"line_number":234,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"9a89bdaa_282c1d98","line":234,"updated":"2016-09-08 08:21:27.000000000","message":"No tailing new line here.","commit_id":"f908fb280fb8ee6f1d22accea39988b7a0b78162"}]}