)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1994,"name":"Zhongyue Luo","email":"zhongyue.luo@gmail.com","username":"zyluo"},"unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2012-12-14 17:40:00 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixed the security group port binding should be automatically"},{"line_number":8,"context_line":"deleted when delete_port"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Added the CASCADE setting for the security group port"},{"line_number":11,"context_line":"binding database model."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAMX%2F%2FzOQ%3D","line":8,"updated":"2012-12-16 04:32:40.000000000","message":"It would be best if the length of lines 7 and 8 did not exceed 50. Look at http://wiki.openstack.org/GerritJenkinsGithub#Gerrit_Best_Practices","commit_id":"9ca602edc465bbef57957a153044f1acfef84682"},{"author":{"_account_id":447,"name":"dan wendlandt","email":"dan@nicira.com","username":"danwent"},"unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2012-12-14 17:40:00 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fixed the security group port binding should be automatically"},{"line_number":8,"context_line":"deleted when delete_port"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Added the CASCADE setting for the security group port"},{"line_number":11,"context_line":"binding database model."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAM3%2F%2F%2F9s%3D","line":8,"in_reply_to":"AAAAMX%2F%2FzOQ%3D","updated":"2012-12-16 20:13:59.000000000","message":"yes, a message like this would work: \n\ndelete sec group binding on port delete.","commit_id":"9ca602edc465bbef57957a153044f1acfef84682"}],"quantum/tests/unit/test_extension_security_group.py":[{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"2c12a553e9bf1e97574ae0531f0b43cf2008d096","unresolved":false,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    def delete_port(self, context, id):"},{"line_number":204,"context_line":"        session \u003d context.session"},{"line_number":205,"context_line":"        with session.begin(subtransactions\u003dTrue):"},{"line_number":206,"context_line":"            self._delete_port_security_group_bindings(context, id)"},{"line_number":207,"context_line":"            super(SecurityGroupTestPlugin, self).delete_port(context, id)"},{"line_number":208,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2FzcI%3D","side":"PARENT","line":205,"updated":"2012-12-15 01:00:18.000000000","message":"Do we need this \u0027with\u0027 after we remove following line?\nMaybe, we can delete this overridden method totally.","commit_id":"dbb8ea79b518dffe7189b32a0c99e56009c5ff84"},{"author":{"_account_id":4184,"name":"Jason Zhang","email":"zhesen@nttmcl.com","username":"zzs"},"change_message_id":"c3585fad659a879b446a7d0297021c822b27eb58","unresolved":false,"context_lines":[{"line_number":202,"context_line":""},{"line_number":203,"context_line":"    def delete_port(self, context, id):"},{"line_number":204,"context_line":"        session \u003d context.session"},{"line_number":205,"context_line":"        with session.begin(subtransactions\u003dTrue):"},{"line_number":206,"context_line":"            self._delete_port_security_group_bindings(context, id)"},{"line_number":207,"context_line":"            super(SecurityGroupTestPlugin, self).delete_port(context, id)"},{"line_number":208,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2Fzbk%3D","side":"PARENT","line":205,"in_reply_to":"AAAAMX%2F%2FzcI%3D","updated":"2012-12-15 01:31:33.000000000","message":"Nice catch! I will remove this.","commit_id":"dbb8ea79b518dffe7189b32a0c99e56009c5ff84"},{"author":{"_account_id":4184,"name":"Jason Zhang","email":"zhesen@nttmcl.com","username":"zzs"},"change_message_id":"c3585fad659a879b446a7d0297021c822b27eb58","unresolved":false,"context_lines":[{"line_number":203,"context_line":"    def delete_port(self, context, id):"},{"line_number":204,"context_line":"        session \u003d context.session"},{"line_number":205,"context_line":"        with session.begin(subtransactions\u003dTrue):"},{"line_number":206,"context_line":"            self._delete_port_security_group_bindings(context, id)"},{"line_number":207,"context_line":"            super(SecurityGroupTestPlugin, self).delete_port(context, id)"},{"line_number":208,"context_line":""},{"line_number":209,"context_line":"    def create_network(self, context, network):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2FzbE%3D","side":"PARENT","line":206,"updated":"2012-12-15 01:37:32.000000000","message":"Without the cascade configured in database model, the test cases will fail If we removed this line. so I think current test cases are enough. :-)","commit_id":"dbb8ea79b518dffe7189b32a0c99e56009c5ff84"}]}