)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17776,"name":"Reedip","email":"reedip.banerjee@gmail.com","username":"Reedip"},"change_message_id":"0d0096538bfcde6a120de31aeeafca4d9907d562","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Kumar Acharya \u003cma501v@intl.att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-06-26 23:48:44 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Closes-Bug: 1593127"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"this fix will allow the delete event to have the required data in the notification payload"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_df84f95e","line":7,"updated":"2016-06-27 09:18:03.000000000","message":"As suggested, please put Closes-Bug before the Change-ID\nAnd also, please limit the message body to 79 characters in one line!","commit_id":"f64a9d196d84076de9bc4af3dcf9238bf2869de6"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"e1c719995b7a5f55231891f4e29cc62629634d8b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Kumar Acharya \u003cma501v@intl.att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-06-26 23:48:44 +0530"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Closes-Bug: 1593127"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"this fix will allow the delete event to have the required data in the notification payload"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_c5f0a769","line":7,"in_reply_to":"3aaa91ec_df84f95e","updated":"2016-06-27 11:06:15.000000000","message":"According to https://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure, I think it should be 72 chars one line.","commit_id":"f64a9d196d84076de9bc4af3dcf9238bf2869de6"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"fbbce22fc64650f2a0b48b7747a65ecad3f3578a","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Kumar Acharya \u003cma501v@intl.att.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-06-28 11:03:21 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"This fix will allow the delete event to have the required data"},{"line_number":8,"context_line":"in the notification payload"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Closes-Bug: 1593127"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3aaa91ec_9313450d","line":7,"updated":"2016-06-28 11:13:35.000000000","message":"The first line should within 50 chars.","commit_id":"48e4e25f95ac6c022f9bf1b40e1dd0d0f05e5657"}],"neutron/api/v2/base.py":[{"author":{"_account_id":14615,"name":"Sergey Belous","email":"belous.py@gmail.com","username":"sbelous"},"change_message_id":"aaf37389c12ba9fefdc767ff6a54cf8d268b76a9","unresolved":false,"context_lines":[{"line_number":545,"context_line":"            raise webob.exc.HTTPBadRequest(msg)"},{"line_number":546,"context_line":"        parent_id \u003d kwargs.get(self._parent_id_name)"},{"line_number":547,"context_line":"        obj \u003d self._item(request, id, parent_id\u003dparent_id)"},{"line_number":548,"context_line":"        result \u003d {self._resource: self._view(request.context, obj)}"},{"line_number":549,"context_line":"        result[self._resource + \u0027_id\u0027] \u003d id"},{"line_number":550,"context_line":"        self._notifier.info(request.context,"},{"line_number":551,"context_line":"                             self._resource + \u0027.delete.start\u0027,"},{"line_number":552,"context_line":"                             result)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3aaa91ec_3e526367","line":549,"range":{"start_line":548,"start_character":0,"end_line":549,"end_character":43},"updated":"2016-06-27 08:29:46.000000000","message":"Why not do it in one line while initializing the dictionary? and also, \u0027result\u0027 it\u0027s not good name in this case","commit_id":"f64a9d196d84076de9bc4af3dcf9238bf2869de6"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"fbbce22fc64650f2a0b48b7747a65ecad3f3578a","unresolved":false,"context_lines":[{"line_number":543,"context_line":"        if request.body:"},{"line_number":544,"context_line":"            msg \u003d _(\u0027Request body is not supported in DELETE.\u0027)"},{"line_number":545,"context_line":"            raise webob.exc.HTTPBadRequest(msg)"},{"line_number":546,"context_line":"        parent_id \u003d kwargs.get(self._parent_id_name)"},{"line_number":547,"context_line":"        obj \u003d self._item(request, id, parent_id\u003dparent_id)"},{"line_number":548,"context_line":"        payload \u003d {self._resource: self._view(request.context, obj)}"},{"line_number":549,"context_line":"        payload[self._resource + \u0027_id\u0027] \u003d id"},{"line_number":550,"context_line":"        self._notifier.info(request.context,"},{"line_number":551,"context_line":"                             self._resource + \u0027.delete.start\u0027,"},{"line_number":552,"context_line":"                             payload)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3aaa91ec_3376d13e","line":549,"range":{"start_line":546,"start_character":0,"end_line":549,"end_character":44},"updated":"2016-06-28 11:13:35.000000000","message":"The bug reports for VIP deletion, but you are changing the basic method for all resources... Is it possible to only change the the bug related code? For example, just change VIP deletion code to add more infor in payload.","commit_id":"48e4e25f95ac6c022f9bf1b40e1dd0d0f05e5657"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"2d9a354f2bdcb1f5cf9e0b67bf50ff1f0f9a2f3b","unresolved":false,"context_lines":[{"line_number":544,"context_line":"            msg \u003d _(\u0027Request body is not supported in DELETE.\u0027)"},{"line_number":545,"context_line":"            raise webob.exc.HTTPBadRequest(msg)"},{"line_number":546,"context_line":"        parent_id \u003d kwargs.get(self._parent_id_name)"},{"line_number":547,"context_line":"        obj \u003d self._item(request, id, parent_id\u003dparent_id)"},{"line_number":548,"context_line":"        payload \u003d {self._resource: self._view(request.context, obj)}"},{"line_number":549,"context_line":"        payload[self._resource + \u0027_id\u0027] \u003d id"},{"line_number":550,"context_line":"        self._notifier.info(request.context,"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_5a6f9bc3","line":547,"updated":"2016-06-30 18:28:16.000000000","message":"all these _item() and _view() are not free, you\u0027re creating overhead in the delete() operation as you\u0027re doing. You should change .end, .start is fine.","commit_id":"df114a250c960f587d0f02c53b8dbad9a37b4d73"}]}