)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"c7fc134a245b42003ae9332aebc902d7c27efa65","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unused PolicyFileNotFound exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"I could not find any use of this exception in Neutron,"},{"line_number":10,"context_line":"nor in any other project. It has a bad message: the policy"},{"line_number":11,"context_line":"file name is hard coded. The policy file name is a config"},{"line_number":12,"context_line":"option so it could be anything."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a061dce_bebfcfa8","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":53},"updated":"2016-04-07 02:55:37.000000000","message":"nit: It looks like we forgot to remove it in the patch[1]. Can you add the description with the commit id in CommitMessage so that we remove with confidence?\n\n[1]: https://review.openstack.org/#/c/123673/","commit_id":"a95b6b60a7d9934a104c72dc0706b30e503444ff"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"18ec129b2b486c8282b31c4b8492add9a6d50786","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unused PolicyFileNotFound exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"I could not find any use of this exception in Neutron,"},{"line_number":10,"context_line":"nor in any other project. It has a bad message: the policy"},{"line_number":11,"context_line":"file name is hard coded. The policy file name is a config"},{"line_number":12,"context_line":"option so it could be anything."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a061dce_28b25c06","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":53},"in_reply_to":"9a061dce_1286a7da","updated":"2016-04-07 08:29:13.000000000","message":"I think it\u0027s not needed to update commit message: it should be enough that you found and pointed the commit which remover the last usage of the exception. Now I\u0027m confident to +2. Thanks!","commit_id":"a95b6b60a7d9934a104c72dc0706b30e503444ff"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"9bd8bf8aa415d1b65ff07ff5ae15379f6dc16129","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unused PolicyFileNotFound exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"I could not find any use of this exception in Neutron,"},{"line_number":10,"context_line":"nor in any other project. It has a bad message: the policy"},{"line_number":11,"context_line":"file name is hard coded. The policy file name is a config"},{"line_number":12,"context_line":"option so it could be anything."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a061dce_a8faec40","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":53},"in_reply_to":"9a061dce_28b25c06","updated":"2016-04-07 08:36:31.000000000","message":"I\u0027m OK.","commit_id":"a95b6b60a7d9934a104c72dc0706b30e503444ff"},{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"f101c5ed47a08eccb165b49f4552e3662ddcef32","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unused PolicyFileNotFound exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"I could not find any use of this exception in Neutron,"},{"line_number":10,"context_line":"nor in any other project. It has a bad message: the policy"},{"line_number":11,"context_line":"file name is hard coded. The policy file name is a config"},{"line_number":12,"context_line":"option so it could be anything."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a061dce_1286a7da","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":53},"in_reply_to":"9a061dce_57af7129","updated":"2016-04-07 07:58:49.000000000","message":"I think that it\u0027s better because I thought \"why did unused code exist?\" and \"Is was on purpose?\" when I looked at this patch.","commit_id":"a95b6b60a7d9934a104c72dc0706b30e503444ff"},{"author":{"_account_id":6524,"name":"Henry Gessau","email":"HenryG@gessau.net","username":"gessau"},"change_message_id":"2d42169ae65ca6b42d1dca8788c533ab5cb7d3dd","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unused PolicyFileNotFound exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"I could not find any use of this exception in Neutron,"},{"line_number":10,"context_line":"nor in any other project. It has a bad message: the policy"},{"line_number":11,"context_line":"file name is hard coded. The policy file name is a config"},{"line_number":12,"context_line":"option so it could be anything."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a061dce_9bcac619","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":53},"in_reply_to":"9a061dce_a8faec40","updated":"2016-04-07 11:22:20.000000000","message":"Actually, I was wondering why it existed. Thanks Hirofumi for finding the history.","commit_id":"a95b6b60a7d9934a104c72dc0706b30e503444ff"},{"author":{"_account_id":14605,"name":"vikram.choudhary","email":"vikschw@gmail.com","username":"vikram.choudhary"},"change_message_id":"830dc67c147e4f63528f108d4915816aaac25031","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove unused PolicyFileNotFound exception"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"I could not find any use of this exception in Neutron,"},{"line_number":10,"context_line":"nor in any other project. It has a bad message: the policy"},{"line_number":11,"context_line":"file name is hard coded. The policy file name is a config"},{"line_number":12,"context_line":"option so it could be anything."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9a061dce_57af7129","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":53},"in_reply_to":"9a061dce_bebfcfa8","updated":"2016-04-07 07:42:36.000000000","message":"Is this really required?","commit_id":"a95b6b60a7d9934a104c72dc0706b30e503444ff"}]}