)]}'
{"swift/common/request_helpers.py":[{"author":{"_account_id":5189,"name":"Yuan Zhou","email":"dunk007@gmail.com","username":"yuan-zhou"},"change_message_id":"6b1b5f78e2deb4224c64cf35b0dfd02881e5ab27","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"def get_name_and_placement(request, *args):"},{"line_number":83,"context_line":"    policy_idx \u003d request.headers.get(POLICY_INDEX, \u00270\u0027)"},{"line_number":84,"context_line":"    print \"AAAAAAAAAAAAAAA %s \" % policy_idx"},{"line_number":85,"context_line":"    try:"},{"line_number":86,"context_line":"        policy_idx \u003d int(policy_idx)"},{"line_number":87,"context_line":"    except ValueError:"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAXH%2F%2FYzY%3D","line":84,"updated":"2014-04-22 06:40:02.000000000","message":"is this print for debug?","commit_id":"32104c57b84a8344e81e2c0832ef0fc20b0598a7"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"60b3a3a7ce5d308d26277ae65b7073bc18f400b5","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"def get_name_and_placement(request, *args):"},{"line_number":83,"context_line":"    policy_idx \u003d request.headers.get(POLICY_INDEX, \u00270\u0027)"},{"line_number":84,"context_line":"    print \"AAAAAAAAAAAAAAA %s \" % policy_idx"},{"line_number":85,"context_line":"    try:"},{"line_number":86,"context_line":"        policy_idx \u003d int(policy_idx)"},{"line_number":87,"context_line":"    except ValueError:"}],"source_content_type":"text/x-python","patch_set":8,"id":"AAAAXH%2F%2FVzg%3D","line":84,"in_reply_to":"AAAAXH%2F%2FYzY%3D","updated":"2014-04-22 12:30:14.000000000","message":"oops yes,  must have been a late night :)","commit_id":"32104c57b84a8344e81e2c0832ef0fc20b0598a7"}],"swift/container/sync.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"368b054aa6ae3414a090141adff2928fed36c139","unresolved":false,"context_lines":[{"line_number":371,"context_line":"                    try:"},{"line_number":372,"context_line":"                        these_headers, this_body \u003d direct_get_object("},{"line_number":373,"context_line":"                            node, part, info[\u0027account\u0027], info[\u0027container\u0027],"},{"line_number":374,"context_line":"                            row[\u0027name\u0027], resp_chunk_size\u003d65536)"},{"line_number":375,"context_line":"                        this_timestamp \u003d float(these_headers[\u0027x-timestamp\u0027])"},{"line_number":376,"context_line":"                        if this_timestamp \u003e timestamp:"},{"line_number":377,"context_line":"                            timestamp \u003d this_timestamp"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAW3%2F%2Fm2I%3D","side":"PARENT","line":374,"updated":"2014-04-11 19:44:21.000000000","message":"don\u0027t you need to send X-Storage-Policy-Index down to the object server?\n\nI think this is why we keep trying to use internal client everywhere we can...","commit_id":"901446938d0743f9834e507cee723856667204ff"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"4c64d245eddde15833b37537a4bd7468fdbb05cc","unresolved":false,"context_lines":[{"line_number":371,"context_line":"                    try:"},{"line_number":372,"context_line":"                        these_headers, this_body \u003d direct_get_object("},{"line_number":373,"context_line":"                            node, part, info[\u0027account\u0027], info[\u0027container\u0027],"},{"line_number":374,"context_line":"                            row[\u0027name\u0027], resp_chunk_size\u003d65536)"},{"line_number":375,"context_line":"                        this_timestamp \u003d float(these_headers[\u0027x-timestamp\u0027])"},{"line_number":376,"context_line":"                        if this_timestamp \u003e timestamp:"},{"line_number":377,"context_line":"                            timestamp \u003d this_timestamp"}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAW3%2F%2Fmsc%3D","side":"PARENT","line":374,"in_reply_to":"AAAAW3%2F%2Fm2I%3D","updated":"2014-04-11 19:58:47.000000000","message":"Need to go read up a bit on direct client but I\u0027m guessing you\u0027re right.  Will let you know if I have questions","commit_id":"901446938d0743f9834e507cee723856667204ff"},{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"4eb1d2fb88965fcb7b0d873351ccf833dbc89d65","unresolved":false,"context_lines":[{"line_number":376,"context_line":"                timestamp \u003d -1"},{"line_number":377,"context_line":"                headers \u003d body \u003d None"},{"line_number":378,"context_line":"                headers_out \u003d {}"},{"line_number":379,"context_line":"                headers_out[POLICY_INDEX] \u003d str(info[\u0027storage_policy_index\u0027])"},{"line_number":380,"context_line":"                for node in nodes:"},{"line_number":381,"context_line":"                    try:"},{"line_number":382,"context_line":"                        these_headers, this_body \u003d direct_get_object("}],"source_content_type":"text/x-python","patch_set":9,"id":"9ad9bd40_2137fc00","line":379,"updated":"2014-05-01 18:21:34.000000000","message":"if i comment this line, all of test_sync still passes","commit_id":"fc47db6e7b1b2760764708478b9d5147ec0df9cb"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"015ffe9253cf127b36ca740f5518e529f6480785","unresolved":false,"context_lines":[{"line_number":376,"context_line":"                timestamp \u003d -1"},{"line_number":377,"context_line":"                headers \u003d body \u003d None"},{"line_number":378,"context_line":"                headers_out \u003d {}"},{"line_number":379,"context_line":"                headers_out[POLICY_INDEX] \u003d str(info[\u0027storage_policy_index\u0027])"},{"line_number":380,"context_line":"                for node in nodes:"},{"line_number":381,"context_line":"                    try:"},{"line_number":382,"context_line":"                        these_headers, this_body \u003d direct_get_object("}],"source_content_type":"text/x-python","patch_set":9,"id":"9ad9bd40_7c68c7ba","line":379,"in_reply_to":"9ad9bd40_2137fc00","updated":"2014-05-01 18:26:08.000000000","message":"I can add a test to specifically check for this but the unit tests won\u0027t be able to do much more than that w/o major work, I think a functional test is required to make sure we can actually sync over containers with different policies","commit_id":"fc47db6e7b1b2760764708478b9d5147ec0df9cb"}],"test/unit/container/test_sync.py":[{"author":{"_account_id":1179,"name":"Clay Gerrard","email":"clay.gerrard@gmail.com","username":"clay-gerrard"},"change_message_id":"368b054aa6ae3414a090141adff2928fed36c139","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    return NewClass"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"@patch_policies([StoragePolicy(0, \u0027zero\u0027, True)])"},{"line_number":91,"context_line":"@patch_rings"},{"line_number":92,"context_line":"class TestContainerSync(unittest.TestCase):"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAW3%2F%2Fm0Q%3D","line":90,"updated":"2014-04-11 19:44:21.000000000","message":"Can you just pass in the FakeRing when you create the mock policies?  It takes a fourth param...","commit_id":"417b1a0bfa43fc2b4f1051cebab7459e3be64aed"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"4c64d245eddde15833b37537a4bd7468fdbb05cc","unresolved":false,"context_lines":[{"line_number":87,"context_line":"    return NewClass"},{"line_number":88,"context_line":""},{"line_number":89,"context_line":""},{"line_number":90,"context_line":"@patch_policies([StoragePolicy(0, \u0027zero\u0027, True)])"},{"line_number":91,"context_line":"@patch_rings"},{"line_number":92,"context_line":"class TestContainerSync(unittest.TestCase):"},{"line_number":93,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"AAAAW3%2F%2Fms4%3D","line":90,"in_reply_to":"AAAAW3%2F%2Fm0Q%3D","updated":"2014-04-11 19:58:47.000000000","message":"good god, I added that and forgot about it.  was wrapped up in trying to, well, get a class wrapper to work","commit_id":"417b1a0bfa43fc2b4f1051cebab7459e3be64aed"}]}