)]}'
{"ceilometerclient/common/http.py":[{"author":{"_account_id":7763,"name":"Alexei Kornienko","email":"akornienko@mirantis.com","username":"akornienko"},"change_message_id":"f3022e1db25e79bb11899e61db49d16f0241844b","unresolved":false,"context_lines":[{"line_number":225,"context_line":"        resp, body \u003d self.json_request(method, url, **kwargs)"},{"line_number":226,"context_line":"        r \u003d requests.Response()"},{"line_number":227,"context_line":"        r._content \u003d json.dumps(body)"},{"line_number":228,"context_line":"        return r"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def get(self, url, **kwargs):"},{"line_number":231,"context_line":"        return self.client_request(\"GET\", url, **kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F89TY%3D","line":228,"updated":"2013-12-18 12:07:12.000000000","message":"This looks really strange.. You load a json object and then dump it back to string.\nYou should refactor json_request instead.","commit_id":"a4820ef63c9dbd96864211637d86a807ecef22a5"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"ed475d7fd23d5b72e7d21ebb46a0f845c618769d","unresolved":false,"context_lines":[{"line_number":225,"context_line":"        resp, body \u003d self.json_request(method, url, **kwargs)"},{"line_number":226,"context_line":"        r \u003d requests.Response()"},{"line_number":227,"context_line":"        r._content \u003d json.dumps(body)"},{"line_number":228,"context_line":"        return r"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def get(self, url, **kwargs):"},{"line_number":231,"context_line":"        return self.client_request(\"GET\", url, **kwargs)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F8x7E%3D","line":228,"in_reply_to":"AAAATn%2F89TY%3D","updated":"2013-12-20 12:07:56.000000000","message":"It\u0027s workaround, which will be removed as soon as we get to use common HttpClient from Oslo.\n\nMethods of managers from oslo require response class on which you can call json(), but class HTTPResponse has no such method.\nIn order to satisfy all of the methods, in terms of expected response class, I decided to apply this decision.","commit_id":"a4820ef63c9dbd96864211637d86a807ecef22a5"},{"author":{"_account_id":7491,"name":"Viktor Serhieiev","email":"viktor.sergeyev@gmail.com","username":"vsergeyev"},"change_message_id":"2378ba405ade8e8d57f82187bbc1ed3d4691e53b","unresolved":false,"context_lines":[{"line_number":227,"context_line":"        r._content \u003d json.dumps(body)"},{"line_number":228,"context_line":"        return r"},{"line_number":229,"context_line":""},{"line_number":230,"context_line":"    def get(self, url, **kwargs):"},{"line_number":231,"context_line":"        return self.client_request(\"GET\", url, **kwargs)"},{"line_number":232,"context_line":""},{"line_number":233,"context_line":"    def post(self, url, **kwargs):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAT3%2F%2F%2FW0%3D","line":230,"updated":"2013-12-25 10:22:08.000000000","message":"we already have BaseClient class in openstack/common/apiclient/client.py file. Can you please reuse it?","commit_id":"a4820ef63c9dbd96864211637d86a807ecef22a5"}],"ceilometerclient/tests/utils.py":[{"author":{"_account_id":7491,"name":"Viktor Serhieiev","email":"viktor.sergeyev@gmail.com","username":"vsergeyev"},"change_message_id":"2378ba405ade8e8d57f82187bbc1ed3d4691e53b","unresolved":false,"context_lines":[{"line_number":16,"context_line":"import copy"},{"line_number":17,"context_line":"import fixtures"},{"line_number":18,"context_line":"import json"},{"line_number":19,"context_line":"import requests"},{"line_number":20,"context_line":"import six"},{"line_number":21,"context_line":"import testtools"},{"line_number":22,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAT3%2F%2F%2FWs%3D","line":19,"updated":"2013-12-25 10:22:08.000000000","message":"Imports should be fixed due to pep8 requirements","commit_id":"a4820ef63c9dbd96864211637d86a807ecef22a5"}]}