)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"b2ffde1bc9df20ebead6c0d5c4536fe0b759467f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove AdminRequired usage in flavor"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"as we didn\u0027t use policy check in db layer, we won\u0027t"},{"line_number":10,"context_line":"raise this kind of exception so no need to catch them"},{"line_number":11,"context_line":"now."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a71b18c_2b47fb4b","line":9,"range":{"start_line":9,"start_character":17,"end_line":9,"end_character":41},"updated":"2016-12-05 13:13:16.000000000","message":"Exactly to say, \u0027AdminRequired\u0027 become useless due to the work about removing DB layer hardcode permission check. We didn\u0027t use policy check in db layer before also.","commit_id":"a40fca46a131e02c077af7c133b9fa3310df0c05"},{"author":{"_account_id":6062,"name":"jichenjc","email":"jichenjc@cn.ibm.com","username":"jichenjc"},"change_message_id":"1f71dee18119a3c81bc4f8166050fc310f17eb44","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove AdminRequired usage in flavor"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"as we didn\u0027t use policy check in db layer, we won\u0027t"},{"line_number":10,"context_line":"raise this kind of exception so no need to catch them"},{"line_number":11,"context_line":"now."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a71b18c_8b0d6fea","line":9,"range":{"start_line":9,"start_character":17,"end_line":9,"end_character":41},"in_reply_to":"3a71b18c_2b47fb4b","updated":"2016-12-05 13:19:39.000000000","message":"Done, permission is better than \u0027policy\u0027 here :)","commit_id":"a40fca46a131e02c077af7c133b9fa3310df0c05"},{"author":{"_account_id":5754,"name":"Alex Xu","email":"hejie.xu@intel.com","username":"xuhj"},"change_message_id":"90904eb3075e7d246021be20df5ca5bd18c17789","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove AdminRequired usage in flavor"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"\u0027AdminRequired\u0027 become useless due to the wrok about removing"},{"line_number":10,"context_line":"DB layer hardcode permission check since d6214f12."},{"line_number":11,"context_line":"we won\u0027t raise this kind of exception so no need to catch them"},{"line_number":12,"context_line":"now."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3a71b18c_8bc2a832","line":10,"range":{"start_line":10,"start_character":41,"end_line":10,"end_character":50},"updated":"2016-12-06 07:19:21.000000000","message":"We should link to this patch https://review.openstack.org/#/c/160269/","commit_id":"9b360f06aa32b58dc6e934ec8d5335366d3a0990"}]}