)]}'
{"swift/common/internal_client.py":[{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"4603dc368b07ac4002858a32851cb3577676ea90","unresolved":false,"context_lines":[{"line_number":159,"context_line":"        self.user_agent \u003d user_agent"},{"line_number":160,"context_line":"        self.request_tries \u003d request_tries"},{"line_number":161,"context_line":""},{"line_number":162,"context_line":"    get_object_ring \u003d pipeline_property(\u0027get_object_ring\u0027)"},{"line_number":163,"context_line":"    container_ring \u003d pipeline_property(\u0027container_ring\u0027)"},{"line_number":164,"context_line":"    account_ring \u003d pipeline_property(\u0027account_ring\u0027)"},{"line_number":165,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FuEg%3D","line":162,"updated":"2014-04-18 17:18:41.000000000","message":"so purely for my own continuing Python education, why place there here like this?  As opposed to either module level functions above where the class is defined or class level functions.  In fact, what are these, just regular module level functions?","commit_id":"5a74ae6685e2b4a2d317289eafa9e205dfefd4d7"}],"test/unit/__init__.py":[{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"1edacf7c526bdcaf39b8fe30a72bd1da7b8a953e","unresolved":false,"context_lines":[{"line_number":296,"context_line":"        rmtree(tempdir)"},{"line_number":297,"context_line":""},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"def with_tempdir(f):"},{"line_number":300,"context_line":"    \"\"\""},{"line_number":301,"context_line":"    Decorator to give a single test a tempdir as argument to test method."},{"line_number":302,"context_line":"    \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FuMU%3D","line":299,"updated":"2014-04-17 22:24:34.000000000","message":"I think you added this for test_backend.py, maybe have it use this one as part of this patch?","commit_id":"5a74ae6685e2b4a2d317289eafa9e205dfefd4d7"}],"test/unit/common/test_internal_client.py":[{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"4603dc368b07ac4002858a32851cb3577676ea90","unresolved":false,"context_lines":[{"line_number":212,"context_line":"        write_fake_ring(account_ring_path)"},{"line_number":213,"context_line":"        container_ring_path \u003d os.path.join(tempdir, \u0027container.ring.gz\u0027)"},{"line_number":214,"context_line":"        write_fake_ring(container_ring_path)"},{"line_number":215,"context_line":"        object_ring_path \u003d os.path.join(tempdir, \u0027object.ring.gz\u0027)"},{"line_number":216,"context_line":"        write_fake_ring(object_ring_path)"},{"line_number":217,"context_line":"        client \u003d internal_client.InternalClient(conf_path, \u0027test\u0027, 1)"},{"line_number":218,"context_line":"        self.assertEqual(client.account_ring, client.app.app.app.account_ring)"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXH%2F%2FocQ%3D","line":215,"updated":"2014-04-18 17:18:41.000000000","message":"if this is proposed to feature/ec, why not patch a few policies in here and test w/more than one obj ring?","commit_id":"5a74ae6685e2b4a2d317289eafa9e205dfefd4d7"}]}