)]}'
{"doc/source/service-api.rst":[{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"5e05501e1615dfea82639c48f085eeb7cd385640","unresolved":false,"context_lines":[{"line_number":209,"context_line":""},{"line_number":210,"context_line":"    ``checksum``: ``True``"},{"line_number":211,"context_line":"        Affects uploads and downloads, and means if check md5 for downloads/uploads."},{"line_number":212,"context_line":""},{"line_number":213,"context_line":"    ``shuffle``: ``False``"},{"line_number":214,"context_line":"        When downloading objects, the default behaviour of the CLI is to shuffle"},{"line_number":215,"context_line":"        lists of objects in order to spread the load on storage drives when multiple"}],"source_content_type":"text/x-rst","patch_set":2,"id":"7aa08908_b21adf6c","line":212,"updated":"2016-06-06 06:46:22.000000000","message":"Good catch. --ignore-checksum option in the shell and checksum option in SwiftService has already existed since 13780f37c3f5e5f18f10f131d1ef39c010457e87","commit_id":"69bf4634b972ef2ee0ec1f015d71223203f0bb1a"}],"swiftclient/service.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4daa70e63dda20b23c424d65432fb2df8d2985fe","unresolved":false,"context_lines":[{"line_number":201,"context_line":"    \u0027dir_marker\u0027: False,"},{"line_number":202,"context_line":"    \u0027checksum\u0027: True,"},{"line_number":203,"context_line":"    \u0027shuffle\u0027: False"},{"line_number":204,"context_line":"}"},{"line_number":205,"context_line":""},{"line_number":206,"context_line":"POLICY \u003d \u0027X-Storage-Policy\u0027"},{"line_number":207,"context_line":"KNOWN_DIR_MARKERS \u003d ("}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_f3f30c71","line":204,"updated":"2016-06-01 17:47:14.000000000","message":"If we add a new option, we should add a default for it here. It would also remove the need to check for\n\n \u0027no_md5check\u0027 in options\n\nHowever, I think I\u0027d prefer to re-use the \u0027checksum\u0027 option that\u0027s already used for uploads.","commit_id":"c60f32aa75144541a988cb087e30862c08839759"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4daa70e63dda20b23c424d65432fb2df8d2985fe","unresolved":false,"context_lines":[{"line_number":984,"context_line":"                                \u0027out_file\u0027: None,"},{"line_number":985,"context_line":"                                \u0027remove_prefix\u0027: False,"},{"line_number":986,"context_line":"                                \u0027shuffle\u0027 : False"},{"line_number":987,"context_line":"                            }"},{"line_number":988,"context_line":""},{"line_number":989,"context_line":"        :returns: A generator for returning the results of the download"},{"line_number":990,"context_line":"                  operations. Each result yielded from the generator is a"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_73cc3c89","line":987,"updated":"2016-06-01 17:47:14.000000000","message":"We should include the option here.","commit_id":"c60f32aa75144541a988cb087e30862c08839759"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4daa70e63dda20b23c424d65432fb2df8d2985fe","unresolved":false,"context_lines":[{"line_number":1137,"context_line":"            headers_receipt \u003d time()"},{"line_number":1138,"context_line":""},{"line_number":1139,"context_line":"            obj_body \u003d _SwiftReader(path, body, headers,"},{"line_number":1140,"context_line":"                                    options[\u0027no_md5check\u0027] if \u0027no_md5check\u0027"},{"line_number":1141,"context_line":"                                    in options else False)"},{"line_number":1142,"context_line":""},{"line_number":1143,"context_line":"            no_file \u003d options[\u0027no_download\u0027]"},{"line_number":1144,"context_line":"            if out_file \u003d\u003d \"-\" and not no_file:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_b3e1842c","line":1141,"range":{"start_line":1140,"start_character":36,"end_line":1141,"end_character":57},"updated":"2016-06-01 17:47:14.000000000","message":"Or equivalently,\n\n options.get(\u0027no_md5check\u0027, False)","commit_id":"c60f32aa75144541a988cb087e30862c08839759"}],"swiftclient/shell.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4daa70e63dda20b23c424d65432fb2df8d2985fe","unresolved":false,"context_lines":[{"line_number":209,"context_line":"                      [--skip-identical] [--remove-prefix]"},{"line_number":210,"context_line":"                      [--header \u003cheader:value\u003e] [--no-shuffle]"},{"line_number":211,"context_line":"                      \u003ccontainer\u003e \u003cobject\u003e"},{"line_number":212,"context_line":"\u0027\u0027\u0027"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"st_download_help \u003d \u0027\u0027\u0027"},{"line_number":215,"context_line":"Download objects from containers."}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_f62bfa7f","line":212,"updated":"2016-06-01 17:47:14.000000000","message":"We\u0027ll need to include the new option here.","commit_id":"c60f32aa75144541a988cb087e30862c08839759"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4daa70e63dda20b23c424d65432fb2df8d2985fe","unresolved":false,"context_lines":[{"line_number":259,"context_line":"                        script to multiple servers). Enable this option to"},{"line_number":260,"context_line":"                        submit download jobs to the thread pool in the order"},{"line_number":261,"context_line":"                        they are listed in the object store."},{"line_number":262,"context_line":"\u0027\u0027\u0027.strip(\"\\n\")"},{"line_number":263,"context_line":""},{"line_number":264,"context_line":""},{"line_number":265,"context_line":"def st_download(parser, args, output_manager):"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_360bc225","line":262,"updated":"2016-06-01 17:47:14.000000000","message":"We\u0027ll need to include the new option here.","commit_id":"c60f32aa75144541a988cb087e30862c08839759"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4daa70e63dda20b23c424d65432fb2df8d2985fe","unresolved":false,"context_lines":[{"line_number":309,"context_line":"        default\u003dFalse, help\u003d\u0027Skip downloading files that are identical on \u0027"},{"line_number":310,"context_line":"        \u0027both sides.\u0027)"},{"line_number":311,"context_line":"    parser.add_argument("},{"line_number":312,"context_line":"        \u0027--no-md5check\u0027, action\u003d\u0027store_true\u0027, dest\u003d\u0027no_md5check\u0027,"},{"line_number":313,"context_line":"        default\u003dFalse, help\u003d\u0027Not check md5\u0027)"},{"line_number":314,"context_line":"    parser.add_argument("},{"line_number":315,"context_line":"        \u0027--no-shuffle\u0027, action\u003d\u0027store_false\u0027, dest\u003d\u0027shuffle\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_36110205","line":312,"range":{"start_line":312,"start_character":11,"end_line":312,"end_character":22},"updated":"2016-06-01 17:47:14.000000000","message":"There\u0027s a similar option for uploads: --ignore-checksum. For the sake of consistency, let\u0027s use the same name here. Look down around line 850 or so.","commit_id":"c60f32aa75144541a988cb087e30862c08839759"}],"tests/unit/test_shell.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"f2b687f4bd2359e8b65c70f5af9bf81673883aaa","unresolved":false,"context_lines":[{"line_number":376,"context_line":"        ]"},{"line_number":377,"context_line":"        with mock.patch(BUILTIN_OPEN) as mock_open, mock.patch("},{"line_number":378,"context_line":"                \u0027swiftclient.service._SwiftReader\u0027) as sr:"},{"line_number":379,"context_line":"            argv \u003d [\"\", \"download\", \"container\", \"object\", \"--ignore-check\"]"},{"line_number":380,"context_line":"            swiftclient.shell.main(argv)"},{"line_number":381,"context_line":"        connection.return_value.get_object.assert_called_with("},{"line_number":382,"context_line":"            \u0027container\u0027, \u0027object\u0027, headers\u003d{}, resp_chunk_size\u003d65536,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9abb7d3a_ee8f63ca","line":379,"range":{"start_line":379,"start_character":60,"end_line":379,"end_character":74},"updated":"2016-06-02 16:31:08.000000000","message":"nit: Technically --ignore-checksum, but argparse will handle the partial matching.","commit_id":"69bf4634b972ef2ee0ec1f015d71223203f0bb1a"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"5e05501e1615dfea82639c48f085eeb7cd385640","unresolved":false,"context_lines":[{"line_number":376,"context_line":"        ]"},{"line_number":377,"context_line":"        with mock.patch(BUILTIN_OPEN) as mock_open, mock.patch("},{"line_number":378,"context_line":"                \u0027swiftclient.service._SwiftReader\u0027) as sr:"},{"line_number":379,"context_line":"            argv \u003d [\"\", \"download\", \"container\", \"object\", \"--ignore-check\"]"},{"line_number":380,"context_line":"            swiftclient.shell.main(argv)"},{"line_number":381,"context_line":"        connection.return_value.get_object.assert_called_with("},{"line_number":382,"context_line":"            \u0027container\u0027, \u0027object\u0027, headers\u003d{}, resp_chunk_size\u003d65536,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_6dda587a","line":379,"range":{"start_line":379,"start_character":60,"end_line":379,"end_character":74},"in_reply_to":"9abb7d3a_ee8f63ca","updated":"2016-06-06 06:46:22.000000000","message":"Wooo, good info. And it looks we can avoid the abbreviation since py35 :P","commit_id":"69bf4634b972ef2ee0ec1f015d71223203f0bb1a"}]}
