)]}'
{".mailmap":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"d28defcd6dc5cc717e59c77e602e6b5d14115b12","unresolved":false,"context_lines":[{"line_number":84,"context_line":"Nakagawa Masaaki \u003cnakagawamsa@nttdata.co.jp\u003e nakagawamsa"},{"line_number":85,"context_line":"Romain Le Disez \u003cromain.ledisez@ovh.net\u003e Romain LE DISEZ"},{"line_number":86,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hpe.com\u003e"},{"line_number":87,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hp.com\u003e"},{"line_number":88,"context_line":"Eamonn O\u0027Toole \u003ceamonn.otoole@hpe.com\u003e \u003ceamonn.otoole@hp.com\u003e"},{"line_number":89,"context_line":"Gerry Drudy \u003cgerry.drudy@hpe.com\u003e \u003cgerry.drudy@hp.com\u003e"},{"line_number":90,"context_line":"Mark Seger \u003cmark.seger@hpe.com\u003e \u003cmark.seger@hp.com\u003e"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_773a9a9a","line":87,"updated":"2017-02-15 10:44:14.000000000","message":"is the a reason why Donagh has been added twice, I mean other then being awesome?","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":84,"context_line":"Nakagawa Masaaki \u003cnakagawamsa@nttdata.co.jp\u003e nakagawamsa"},{"line_number":85,"context_line":"Romain Le Disez \u003cromain.ledisez@ovh.net\u003e Romain LE DISEZ"},{"line_number":86,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hpe.com\u003e"},{"line_number":87,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hp.com\u003e"},{"line_number":88,"context_line":"Eamonn O\u0027Toole \u003ceamonn.otoole@hpe.com\u003e \u003ceamonn.otoole@hp.com\u003e"},{"line_number":89,"context_line":"Gerry Drudy \u003cgerry.drudy@hpe.com\u003e \u003cgerry.drudy@hp.com\u003e"},{"line_number":90,"context_line":"Mark Seger \u003cmark.seger@hpe.com\u003e \u003cmark.seger@hp.com\u003e"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_8dc7e194","line":87,"in_reply_to":"fa31d9ce_773a9a9a","updated":"2017-02-15 17:45:00.000000000","message":"The first maps @hpe.com, second maps hp.com. There\u0027s no way in the syntax of the file to map more than one address to the target in a single line. There\u0027s many other examples of this in this file.","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ff8bfb19603d865391bfa72de475d5aedff8ee2e","unresolved":false,"context_lines":[{"line_number":84,"context_line":"Nakagawa Masaaki \u003cnakagawamsa@nttdata.co.jp\u003e nakagawamsa"},{"line_number":85,"context_line":"Romain Le Disez \u003cromain.ledisez@ovh.net\u003e Romain LE DISEZ"},{"line_number":86,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hpe.com\u003e"},{"line_number":87,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hp.com\u003e"},{"line_number":88,"context_line":"Eamonn O\u0027Toole \u003ceamonn.otoole@hpe.com\u003e \u003ceamonn.otoole@hp.com\u003e"},{"line_number":89,"context_line":"Gerry Drudy \u003cgerry.drudy@hpe.com\u003e \u003cgerry.drudy@hp.com\u003e"},{"line_number":90,"context_line":"Mark Seger \u003cmark.seger@hpe.com\u003e \u003cmark.seger@hp.com\u003e"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_9870010d","line":87,"updated":"2017-02-15 18:27:10.000000000","message":"? I don\u0027t understand this change.","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"43fe88d1ad8451af65fd5e3fbc48965e5cbd5819","unresolved":false,"context_lines":[{"line_number":84,"context_line":"Nakagawa Masaaki \u003cnakagawamsa@nttdata.co.jp\u003e nakagawamsa"},{"line_number":85,"context_line":"Romain Le Disez \u003cromain.ledisez@ovh.net\u003e Romain LE DISEZ"},{"line_number":86,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hpe.com\u003e"},{"line_number":87,"context_line":"Donagh McCabe \u003cdonagh.mccabe@gmail.com\u003e \u003cdonagh.mccabe@hp.com\u003e"},{"line_number":88,"context_line":"Eamonn O\u0027Toole \u003ceamonn.otoole@hpe.com\u003e \u003ceamonn.otoole@hp.com\u003e"},{"line_number":89,"context_line":"Gerry Drudy \u003cgerry.drudy@hpe.com\u003e \u003cgerry.drudy@hp.com\u003e"},{"line_number":90,"context_line":"Mark Seger \u003cmark.seger@hpe.com\u003e \u003cmark.seger@hp.com\u003e"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_656ba1b7","line":87,"in_reply_to":"fa31d9ce_9870010d","updated":"2017-02-15 18:42:49.000000000","message":"When Donagh added himself to \"Core Emeritus\", he used his gmail address and added a mailmap entry to map previous commits to his gmail address. He\u0027s contributed under 3 email addresses (hp, hpe, and gmail). This section maps all of his contributions to one email, and I\u0027m using his last patch as an indicator of his intent to want to be referenced at his gmail account.\n\nNote this has nothing to do with corporate affiliation. It\u0027s just a way to map multiple (name, email) pairs to a single entry. This is how git tracks authors, so it means he\u0027s counted once instead of 3 times.","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"}],"AUTHORS":[{"author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"change_message_id":"825749c748914e9347fcf52c5c41c9d92af0bc51","unresolved":false,"context_lines":[{"line_number":110,"context_line":"Dmitriy Ukhlov (dukhlov@mirantis.com)"},{"line_number":111,"context_line":"Dmitry Ukov (dukov@mirantis.com)"},{"line_number":112,"context_line":"Dolph Mathews (dolph.mathews@gmail.com)"},{"line_number":113,"context_line":"Donagh McCabe (donagh.mccabe@gmail.com)"},{"line_number":114,"context_line":"Doron Chen (cdoron@il.ibm.com)"},{"line_number":115,"context_line":"Doug Hellmann (doug.hellmann@dreamhost.com)"},{"line_number":116,"context_line":"Doug Weimer (dweimer@gmail.com)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_2d6995f9","line":113,"updated":"2017-02-15 18:20:19.000000000","message":"noticed you are adding Donagh and Hisashi\u0027s name back here, not sure if it was on purpose or just a merge issue...they are also in the core emeritus list...","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"43fe88d1ad8451af65fd5e3fbc48965e5cbd5819","unresolved":false,"context_lines":[{"line_number":110,"context_line":"Dmitriy Ukhlov (dukhlov@mirantis.com)"},{"line_number":111,"context_line":"Dmitry Ukov (dukov@mirantis.com)"},{"line_number":112,"context_line":"Dolph Mathews (dolph.mathews@gmail.com)"},{"line_number":113,"context_line":"Donagh McCabe (donagh.mccabe@gmail.com)"},{"line_number":114,"context_line":"Doron Chen (cdoron@il.ibm.com)"},{"line_number":115,"context_line":"Doug Hellmann (doug.hellmann@dreamhost.com)"},{"line_number":116,"context_line":"Doug Weimer (dweimer@gmail.com)"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_d305fa13","line":113,"in_reply_to":"fa31d9ce_2d6995f9","updated":"2017-02-15 18:42:49.000000000","message":"Yes, that was my intention (same with Hisashi). IMO the \"Core Emeritus\" section is to highlight contributors, not for specific contributor acknowledgement. The acknowledgement comes from either the \"Original Authors\" or the \"Contributors\" sections.","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"}],"CHANGELOG":[{"author":{"_account_id":12261,"name":"Mahati","email":"mahati.chamarthy@gmail.com","username":"Mahati"},"change_message_id":"813721ef03a2cba7b6fa3bdd7f9471e29761dd45","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      - Optimized the common case for hashing filesystem trees, thus"},{"line_number":9,"context_line":"        eliminating a lot of extraneous disk I/O."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"      - Updated the `hashes.pkl` file format to include some timestamp"},{"line_number":12,"context_line":"        information for race detection. Also simplified some logic to prevent"},{"line_number":13,"context_line":"        race conditions and optimize for the common case."},{"line_number":14,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_0924e25b","line":11,"range":{"start_line":11,"start_character":56,"end_line":11,"end_character":70},"updated":"2017-02-15 07:18:36.000000000","message":"nit: maybe a bit more definitive by removing \u0027some\u0027 and saying \"to include timestamp information\"?","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":8,"context_line":"      - Optimized the common case for hashing filesystem trees, thus"},{"line_number":9,"context_line":"        eliminating a lot of extraneous disk I/O."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"      - Updated the `hashes.pkl` file format to include some timestamp"},{"line_number":12,"context_line":"        information for race detection. Also simplified some logic to prevent"},{"line_number":13,"context_line":"        race conditions and optimize for the common case."},{"line_number":14,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_ad702573","line":11,"in_reply_to":"fa31d9ce_0924e25b","updated":"2017-02-15 17:45:00.000000000","message":"Done","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"change_message_id":"ae1d9bceb28550eeebb1dc6123d0c7408a633f94","unresolved":false,"context_lines":[{"line_number":9,"context_line":"        eliminating a lot of extraneous disk I/O."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"      - Updated the `hashes.pkl` file format to include some timestamp"},{"line_number":12,"context_line":"        information for race detection. Also simplified some logic to prevent"},{"line_number":13,"context_line":"        race conditions and optimize for the common case."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"      - The erasure code reconstructor will now shuffle work jobs across all"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_fe91ed47","line":12,"range":{"start_line":12,"start_character":56,"end_line":12,"end_character":66},"updated":"2017-02-15 16:28:14.000000000","message":"agree with Mahati, some timestamp and some logic seems \"weirdly vague\" for this kind of document.","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":9,"context_line":"        eliminating a lot of extraneous disk I/O."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"      - Updated the `hashes.pkl` file format to include some timestamp"},{"line_number":12,"context_line":"        information for race detection. Also simplified some logic to prevent"},{"line_number":13,"context_line":"        race conditions and optimize for the common case."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"      - The erasure code reconstructor will now shuffle work jobs across all"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_0d6131a9","line":12,"in_reply_to":"fa31d9ce_fe91ed47","updated":"2017-02-15 17:45:00.000000000","message":"Done","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"change_message_id":"ae1d9bceb28550eeebb1dc6123d0c7408a633f94","unresolved":false,"context_lines":[{"line_number":27,"context_line":"      fragments, Swift will emit a warning. This is a configuration that is"},{"line_number":28,"context_line":"      known to harm data durability. In a future release, this warning will be"},{"line_number":29,"context_line":"      upgraded to an error unless the policy is marked as deprecated. All data"},{"line_number":30,"context_line":"      in an isa_l_rs_vand storage policy should be migrated as soon as"},{"line_number":31,"context_line":"      possible. Please see https://bugs.launchpad.net/swift/+bug/1639691 for"},{"line_number":32,"context_line":"      more information."},{"line_number":33,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_21660451","line":30,"range":{"start_line":30,"start_character":12,"end_line":30,"end_character":40},"updated":"2017-02-15 16:28:14.000000000","message":"should it specify the 5 or more parity too?","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":27,"context_line":"      fragments, Swift will emit a warning. This is a configuration that is"},{"line_number":28,"context_line":"      known to harm data durability. In a future release, this warning will be"},{"line_number":29,"context_line":"      upgraded to an error unless the policy is marked as deprecated. All data"},{"line_number":30,"context_line":"      in an isa_l_rs_vand storage policy should be migrated as soon as"},{"line_number":31,"context_line":"      possible. Please see https://bugs.launchpad.net/swift/+bug/1639691 for"},{"line_number":32,"context_line":"      more information."},{"line_number":33,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_0dd69127","line":30,"in_reply_to":"fa31d9ce_21660451","updated":"2017-02-15 17:45:00.000000000","message":"Done","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"d28defcd6dc5cc717e59c77e602e6b5d14115b12","unresolved":false,"context_lines":[{"line_number":38,"context_line":"      rebalance.  The handoffs_only behavior is somewhat consistent with the"},{"line_number":39,"context_line":"      replicator\u0027s handoffs_first option (any error on any handoff in the"},{"line_number":40,"context_line":"      replicator will make it essentially handoff only forever) but the"},{"line_number":41,"context_line":"      option does what you want and is named correctly in the reconstructor."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    * Temporary URLs now support ISO 8601 timestamps in addition to Unix"},{"line_number":44,"context_line":"      seconds-since-epoch timestamps. The ISO 8601 format accepted is"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_50bcb6ba","line":41,"updated":"2017-02-15 10:44:14.000000000","message":"I think we need to mention it\u0027ll solely focus on rebalance, until it\u0027s disabled. And the \u0027somewhat consistent with the replicators handoff_first option ..\u0027 is cool. But I wonder if it confuses more then it explains. It is great in the git commit message, but wonder if this needs to be reworded a little. Maybe something like:\n\n * The erasure code reconstructor `handoffs_first` option has been\n   deprecated in favor of `handoffs_only`. `handoffs_only` is far more\n   useful, and just like handoffs_first mode in the replicator, it gives\n   the operator the option of forcing the consistency engine to focus solely\n   on revert (handoff) jobs improving the speed of rebalances.  The\n   handoffs_only behavior is somewhat consistent with the replicator\u0027s\n   handoffs_first option, whereby it will focus soley on handoffs before\n   moving onto other jobs, but the option does what you want and is named\n   correctly in the reconstructor.\n\nThough this is potentially just a wording NIT","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":12261,"name":"Mahati","email":"mahati.chamarthy@gmail.com","username":"Mahati"},"change_message_id":"5229b72933e1f56fbe4267cc31032677fbc8794f","unresolved":false,"context_lines":[{"line_number":38,"context_line":"      rebalance.  The handoffs_only behavior is somewhat consistent with the"},{"line_number":39,"context_line":"      replicator\u0027s handoffs_first option (any error on any handoff in the"},{"line_number":40,"context_line":"      replicator will make it essentially handoff only forever) but the"},{"line_number":41,"context_line":"      option does what you want and is named correctly in the reconstructor."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    * Temporary URLs now support ISO 8601 timestamps in addition to Unix"},{"line_number":44,"context_line":"      seconds-since-epoch timestamps. The ISO 8601 format accepted is"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_93a1d848","line":41,"in_reply_to":"fa31d9ce_50bcb6ba","updated":"2017-02-15 10:51:41.000000000","message":"@Matt - \"whereby it will focus soley on handoffs before moving onto other jobs\" - But handoffs_only does not move onto other jobs, it will remain in \"handoffs_only\" mode forever until it is turned off?","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":38,"context_line":"      rebalance.  The handoffs_only behavior is somewhat consistent with the"},{"line_number":39,"context_line":"      replicator\u0027s handoffs_first option (any error on any handoff in the"},{"line_number":40,"context_line":"      replicator will make it essentially handoff only forever) but the"},{"line_number":41,"context_line":"      option does what you want and is named correctly in the reconstructor."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    * Temporary URLs now support ISO 8601 timestamps in addition to Unix"},{"line_number":44,"context_line":"      seconds-since-epoch timestamps. The ISO 8601 format accepted is"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_8d4281ac","line":41,"in_reply_to":"fa31d9ce_50bcb6ba","updated":"2017-02-15 17:45:00.000000000","message":"Done","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":38,"context_line":"      rebalance.  The handoffs_only behavior is somewhat consistent with the"},{"line_number":39,"context_line":"      replicator\u0027s handoffs_first option (any error on any handoff in the"},{"line_number":40,"context_line":"      replicator will make it essentially handoff only forever) but the"},{"line_number":41,"context_line":"      option does what you want and is named correctly in the reconstructor."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    * Temporary URLs now support ISO 8601 timestamps in addition to Unix"},{"line_number":44,"context_line":"      seconds-since-epoch timestamps. The ISO 8601 format accepted is"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_ad3d852c","line":41,"in_reply_to":"fa31d9ce_93a1d848","updated":"2017-02-15 17:45:00.000000000","message":"Done","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"d28defcd6dc5cc717e59c77e602e6b5d14115b12","unresolved":false,"context_lines":[{"line_number":40,"context_line":"      replicator will make it essentially handoff only forever) but the"},{"line_number":41,"context_line":"      option does what you want and is named correctly in the reconstructor."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    * Temporary URLs now support ISO 8601 timestamps in addition to Unix"},{"line_number":44,"context_line":"      seconds-since-epoch timestamps. The ISO 8601 format accepted is"},{"line_number":45,"context_line":"      \u0027%Y-%m-%dT%H:%M:%SZ\u0027. This makes TempURLs more user-friendly to produce"},{"line_number":46,"context_line":"      and consume."}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_f3bdb402","line":43,"range":{"start_line":43,"start_character":34,"end_line":43,"end_character":52},"updated":"2017-02-15 10:44:14.000000000","message":"NIT: ISO 8601 actually describes a large number of date/time formats. And only 1 ISO 8601 form is supported by TempURL.\n\nYou make it more clear below. So I guess it\u0027s ok. But we could say:\n\n  * Temporary URLs now supports 1 common form of ISO 8601 timestamps in addition to Unix..","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":40,"context_line":"      replicator will make it essentially handoff only forever) but the"},{"line_number":41,"context_line":"      option does what you want and is named correctly in the reconstructor."},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"    * Temporary URLs now support ISO 8601 timestamps in addition to Unix"},{"line_number":44,"context_line":"      seconds-since-epoch timestamps. The ISO 8601 format accepted is"},{"line_number":45,"context_line":"      \u0027%Y-%m-%dT%H:%M:%SZ\u0027. This makes TempURLs more user-friendly to produce"},{"line_number":46,"context_line":"      and consume."}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_ed43fdac","line":43,"in_reply_to":"fa31d9ce_f3bdb402","updated":"2017-02-15 17:45:00.000000000","message":"Done","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"change_message_id":"ae1d9bceb28550eeebb1dc6123d0c7408a633f94","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    * Fixed the stats calculation in the erasure code reconstructor."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    * Rings with min_part_hours set to zero will now one move one partition"},{"line_number":74,"context_line":"      replica per rebalance, thus matching behavior when min_part_hours is"},{"line_number":75,"context_line":"      greater than zero."},{"line_number":76,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_4143f86b","line":73,"range":{"start_line":73,"start_character":53,"end_line":73,"end_character":57},"updated":"2017-02-15 16:28:14.000000000","message":"remove??","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"f60b25a07428b9c8ab49bfd5222619de53a61adc","unresolved":false,"context_lines":[{"line_number":70,"context_line":""},{"line_number":71,"context_line":"    * Fixed the stats calculation in the erasure code reconstructor."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    * Rings with min_part_hours set to zero will now one move one partition"},{"line_number":74,"context_line":"      replica per rebalance, thus matching behavior when min_part_hours is"},{"line_number":75,"context_line":"      greater than zero."},{"line_number":76,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"fa31d9ce_8d29a1e8","line":73,"in_reply_to":"fa31d9ce_4143f86b","updated":"2017-02-15 17:45:00.000000000","message":"will now only move one","commit_id":"03e8843a25d6080424967aa346b015d058d09eeb"},{"author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"change_message_id":"825749c748914e9347fcf52c5c41c9d92af0bc51","unresolved":false,"context_lines":[{"line_number":1,"context_line":"swift (2.13.0, OpenStack Ocata)"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"    * Improvements in key parts of the consistency checking systems"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"      - Improved performance by eliminating an unneeded directory"},{"line_number":6,"context_line":"        structure hash."}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_9816414a","line":3,"range":{"start_line":3,"start_character":39,"end_line":3,"end_character":59},"updated":"2017-02-15 18:20:19.000000000","message":"nit: the docs typically refer to \"consistency engine\" (same as line 37. maybe use that here too??","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"43fe88d1ad8451af65fd5e3fbc48965e5cbd5819","unresolved":false,"context_lines":[{"line_number":1,"context_line":"swift (2.13.0, OpenStack Ocata)"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"    * Improvements in key parts of the consistency checking systems"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"      - Improved performance by eliminating an unneeded directory"},{"line_number":6,"context_line":"        structure hash."}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_c55c4d1b","line":3,"range":{"start_line":3,"start_character":39,"end_line":3,"end_character":59},"in_reply_to":"fa31d9ce_9816414a","updated":"2017-02-15 18:42:49.000000000","message":"Done","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":5600,"name":"Brian Cline","email":"bcline@softlayer.com","username":"briancline"},"change_message_id":"c250363f3291d9a4dc5752ae01a7bf591634d5dd","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        until it can be moved to a primary node."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"      Upgrade Impact: If you upgrade and roll back, you must delete all"},{"line_number":24,"context_line":"      `hashed.pkl` files."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    * If using erasure coding with ISA-L in rs_vand mode and 5 or more parity"},{"line_number":27,"context_line":"      fragments, Swift will emit a warning. This is a configuration that is"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_f81addc4","line":24,"updated":"2017-02-15 18:09:39.000000000","message":"*hashes","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"d2777a8beefa55225da38937c253910e17fa5f3f","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        until it can be moved to a primary node."},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"      Upgrade Impact: If you upgrade and roll back, you must delete all"},{"line_number":24,"context_line":"      `hashed.pkl` files."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"    * If using erasure coding with ISA-L in rs_vand mode and 5 or more parity"},{"line_number":27,"context_line":"      fragments, Swift will emit a warning. This is a configuration that is"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_b366c6ea","line":24,"in_reply_to":"fa31d9ce_f81addc4","updated":"2017-02-15 18:16:47.000000000","message":"Done","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ff8bfb19603d865391bfa72de475d5aedff8ee2e","unresolved":false,"context_lines":[{"line_number":32,"context_line":"      https://bugs.launchpad.net/swift/+bug/1639691 for more information."},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"    * The erasure code reconstructor `handoffs_first` option has been"},{"line_number":35,"context_line":"      deprecated in favor of `handoffs_only`. `handoffs_only` is far more"},{"line_number":36,"context_line":"      useful, and just like `handoffs_first` mode in the replicator, it gives"},{"line_number":37,"context_line":"      the operator the option of forcing the consistency engine to focus"},{"line_number":38,"context_line":"      solely on revert (handoff) jobs, thus improving the speed of"},{"line_number":39,"context_line":"      rebalances.  The `handoffs_only` behavior is somewhat consistent with"},{"line_number":40,"context_line":"      the replicator\u0027s `handoffs_first` option (any error on any handoff in"},{"line_number":41,"context_line":"      the replicator will make it essentially handoff only forever) but the"},{"line_number":42,"context_line":"      `handoff_only` option does what you want and is named correctly in the"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_9861e107","line":39,"range":{"start_line":35,"start_character":46,"end_line":39,"end_character":17},"updated":"2017-02-15 18:27:10.000000000","message":"This seems a little long, but I\u0027m not sure how best to break it up.","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ff8bfb19603d865391bfa72de475d5aedff8ee2e","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      containers."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    * The default for `object_post_as_copy` has been changed to False. The"},{"line_number":56,"context_line":"      option is now deprecated and will be removed in a future release."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    * Fixed a bug where the ring builder would not allow removal of a device"},{"line_number":59,"context_line":"      when min_part_seconds_left was greater than zero."}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_9316224a","line":56,"updated":"2017-02-15 18:27:10.000000000","message":"I also feel like this deserves to be higher -- deprecation warning seems to trump new features?","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":9625,"name":"Thiago da Silva","email":"thiagodasilva@gmail.com","username":"thiago"},"change_message_id":"825749c748914e9347fcf52c5c41c9d92af0bc51","unresolved":false,"context_lines":[{"line_number":53,"context_line":"      containers."},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    * The default for `object_post_as_copy` has been changed to False. The"},{"line_number":56,"context_line":"      option is now deprecated and will be removed in a future release."},{"line_number":57,"context_line":""},{"line_number":58,"context_line":"    * Fixed a bug where the ring builder would not allow removal of a device"},{"line_number":59,"context_line":"      when min_part_seconds_left was greater than zero."}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_f8ec9dbc","line":56,"updated":"2017-02-15 18:20:19.000000000","message":"I wonder if we should have a stronger message encouraging operators to change to fast post as future features will not support post_as_copy.","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ff8bfb19603d865391bfa72de475d5aedff8ee2e","unresolved":false,"context_lines":[{"line_number":70,"context_line":"      ETag header of the md5 sum of the concatenated md5 sums of the"},{"line_number":71,"context_line":"      referenced segments."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    * Fixed the stats calculation in the erasure code reconstructor."},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    * Rings with min_part_hours set to zero will now only move one partition"},{"line_number":76,"context_line":"      replica per rebalance, thus matching behavior when min_part_hours is"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_537bca7a","line":73,"updated":"2017-02-15 18:27:10.000000000","message":"Which commit was this? If it was mine, that was mainly fallout from Clay\u0027s disk-shuffling -- it no longer made sense to log the disk completion percentage. Not sure it would warrant a release note.","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"43fe88d1ad8451af65fd5e3fbc48965e5cbd5819","unresolved":false,"context_lines":[{"line_number":70,"context_line":"      ETag header of the md5 sum of the concatenated md5 sums of the"},{"line_number":71,"context_line":"      referenced segments."},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    * Fixed the stats calculation in the erasure code reconstructor."},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"    * Rings with min_part_hours set to zero will now only move one partition"},{"line_number":76,"context_line":"      replica per rebalance, thus matching behavior when min_part_hours is"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_05d6654e","line":73,"in_reply_to":"fa31d9ce_537bca7a","updated":"2017-02-15 18:42:49.000000000","message":"Nope, not yours. https://review.openstack.org/#/c/283946/","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"ff8bfb19603d865391bfa72de475d5aedff8ee2e","unresolved":false,"context_lines":[{"line_number":76,"context_line":"      replica per rebalance, thus matching behavior when min_part_hours is"},{"line_number":77,"context_line":"      greater than zero."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    * I/O priority is now supported on AArch64 architecture."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    * Various other minor bug fixes and improvements."},{"line_number":82,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_b37a4622","line":79,"updated":"2017-02-15 18:27:10.000000000","message":"I *thought* Clay was kidding about needing to include this one.\n\nDo we ever explicitly state which platforms we support? \u0027Cause really, it\u0027s just x86_64 and anything else that may work does so purely by accident -- if deployers want or need more architectures supported, they need to engage with the community so we can all be confident that\n\n* the current code really *does* work on that architecture, and\n* future changes aren\u0027t going to *break* support for that architecture.","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"},{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"43fe88d1ad8451af65fd5e3fbc48965e5cbd5819","unresolved":false,"context_lines":[{"line_number":76,"context_line":"      replica per rebalance, thus matching behavior when min_part_hours is"},{"line_number":77,"context_line":"      greater than zero."},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"    * I/O priority is now supported on AArch64 architecture."},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"    * Various other minor bug fixes and improvements."},{"line_number":82,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"fa31d9ce_e584f15e","line":79,"in_reply_to":"fa31d9ce_b37a4622","updated":"2017-02-15 18:42:49.000000000","message":"I could alternatively say \"fixed a bug where I/O priority was broken on AArch64 architecture.\"","commit_id":"cd6d787b0ca5d85064faa147141f2ccdba0c8963"}]}