)]}'
{"etc/policy.json":[{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"fcf7cbe6cc2a7f63927b31ca1362fc79272f72c1","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    \"protectable:get\": \"rule:admin_or_owner\","},{"line_number":20,"context_line":"    \"protectable:get_all\": \"rule:admin_or_owner\","},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    \"provider:get\": \"rule:admin_api\","},{"line_number":23,"context_line":"    \"provider:get_all\": \"rule:admin_api\""},{"line_number":24,"context_line":"}"}],"source_content_type":"application/json","patch_set":2,"id":"7af24918_1169cc8b","line":22,"range":{"start_line":22,"start_character":21,"end_line":22,"end_character":35},"updated":"2016-03-02 17:57:06.000000000","message":"I think that this should be rule:admin_or_owner","commit_id":"0ebc931286869f1f52fe62c1de4846c920564249"},{"author":{"_account_id":13070,"name":"Eran Gampel","email":"eran@gampel.net","username":"Gampel"},"change_message_id":"fcf7cbe6cc2a7f63927b31ca1362fc79272f72c1","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    \"protectable:get_all\": \"rule:admin_or_owner\","},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    \"provider:get\": \"rule:admin_api\","},{"line_number":23,"context_line":"    \"provider:get_all\": \"rule:admin_api\""},{"line_number":24,"context_line":"}"}],"source_content_type":"application/json","patch_set":2,"id":"7af24918_d1f5a4cb","line":23,"range":{"start_line":23,"start_character":25,"end_line":23,"end_character":40},"updated":"2016-03-02 17:57:06.000000000","message":"rule:admin_or_owner","commit_id":"0ebc931286869f1f52fe62c1de4846c920564249"}],"smaug/services/protection/api.py":[{"author":{"_account_id":16203,"name":"zengyingzhe","email":"zengyingzhe@huawei.com","username":"zengyingzhe"},"change_message_id":"3a3ef8e637e4137a05d4c2b3f8811bc8d08d99fd","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                       sort_keys, sort_dirs, filters, offset):"},{"line_number":63,"context_line":"        return self.protection_rpcapi.\\"},{"line_number":64,"context_line":"            list_providers(context, marker, limit,"},{"line_number":65,"context_line":"                           sort_keys, sort_dirs, filters)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aed3d3a_780f4088","line":65,"updated":"2016-02-29 07:35:36.000000000","message":"where is offset parameter used?\nIf it\u0027s not necessary, please remove it from arg list.","commit_id":"ad50a0088c5fa5c8c33ff1241899385f239f5630"},{"author":{"_account_id":17151,"name":"chenying","email":"chenyingko@gmail.com","username":"chenying"},"change_message_id":"3a35253436cbc296b68d85f3038f7fa0788459f7","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                       sort_keys, sort_dirs, filters, offset):"},{"line_number":63,"context_line":"        return self.protection_rpcapi.\\"},{"line_number":64,"context_line":"            list_providers(context, marker, limit,"},{"line_number":65,"context_line":"                           sort_keys, sort_dirs, filters)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aed3d3a_9b000368","line":65,"in_reply_to":"9aed3d3a_780f4088","updated":"2016-03-01 02:17:21.000000000","message":"https://review.openstack.org/#/c/280051/1..5/smaug/common/sqlalchemyutils.py\nNow we are using the paginate_query module of oslo_db/sqlalchemy/utils.py. offset is not supported in the version. But other serveral porject have introduce offset to paginate_query. So I keep parameter offset here. After oslo_db/sqlalchemy adding this parameter , I will using it in \npaginate query.","commit_id":"ad50a0088c5fa5c8c33ff1241899385f239f5630"}]}