)]}'
{"ceilometerclient/v2/query.py":[{"author":{"_account_id":2284,"name":"Eoghan Glynn","email":"eglynn@redhat.com","username":"eglynn"},"change_message_id":"7475bc8acb4d65fc5f07121b9a52dd4070dc3a7b","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        if limit:"},{"line_number":31,"context_line":"            query[\"limit\"] \u003d limit"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        url \u003d self._path()"},{"line_number":34,"context_line":"        resp, body \u003d self.api.json_request(\u0027POST\u0027,"},{"line_number":35,"context_line":"                                           url,"},{"line_number":36,"context_line":"                                           body\u003dquery)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FXFk%3D","line":33,"updated":"2014-03-18 11:35:29.000000000","message":"How about restructuring this code to use the same pattern to patermeterize the path as is already used for the resource class, for example:\n\n  class QueryManager(base.Manager):\n      path_suffix \u003d None\n\n      def query(self, filter, orderby, limit):\n          ...\n          url \u003d \u0027/v2/query/%s\u0027 % self.path_suffix\n          ...\n\n  class QuerySamplesManager(QueryManager):\n      resource_class \u003d samples.Sample\n      path_suffix \u003d \u0027samples\u0027\n\n\n  class QueryAlarmManager(QueryManager):\n      resource_class \u003d alarms.Alarm\n      path_suffix \u003d \u0027alarms\u0027","commit_id":"f29f6803fed019411cc909f8ecbd562ef6b9ae12"},{"author":{"_account_id":9562,"name":"Ildiko Vancsa","email":"ildiko.vancsa@gmail.com","username":"ildikov"},"change_message_id":"34e79cee52e4dc2b7401bc09af913ab285532e63","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        if limit:"},{"line_number":31,"context_line":"            query[\"limit\"] \u003d limit"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"        url \u003d self._path()"},{"line_number":34,"context_line":"        resp, body \u003d self.api.json_request(\u0027POST\u0027,"},{"line_number":35,"context_line":"                                           url,"},{"line_number":36,"context_line":"                                           body\u003dquery)"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FV24%3D","line":33,"in_reply_to":"AAAAWH%2F%2FXFk%3D","updated":"2014-03-18 14:15:11.000000000","message":"Done","commit_id":"f29f6803fed019411cc909f8ecbd562ef6b9ae12"}]}
