)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"e892ce3188dcbb4e11e5711878159040f58973b9","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Update Erasure Coding Overview doc to remove Beta version"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The major functionality of EC has been released for Liberty and"},{"line_number":10,"context_line":"the beta version of the code has changed to \"transition to"},{"line_number":11,"context_line":"production\" period.  It is with caution that deployers use the"},{"line_number":12,"context_line":"erasure coding storage policy for producation at large scale."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3a7e1126_4d1e6e60","line":10,"updated":"2015-12-18 15:56:24.000000000","message":"edit wording according to my other comment","commit_id":"2ce5ba94339a2ac77b126069379b92a3465e02eb"}],"doc/source/overview_erasure_code.rst":[{"author":{"_account_id":330,"name":"John Dickinson","email":"me@not.mn","username":"notmyname"},"change_message_id":"e1df2fdec4f312ab911218aaa99b234818069d68","unresolved":false,"context_lines":[{"line_number":9,"context_line":"The erasure code support in Swift is considered \"ready for production\"."},{"line_number":10,"context_line":"The major functionality is included, but we need to continue to test"},{"line_number":11,"context_line":"or validate erasure code runs at large scale. Deployers are urged to do"},{"line_number":12,"context_line":"extensive testing and run production data using an erasure code storage"},{"line_number":13,"context_line":"policy."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"If any bugs are found during testing, please report them to"}],"source_content_type":"text/x-rst","patch_set":1,"id":"ba8a016a_821c916b","line":12,"updated":"2015-11-20 18:15:47.000000000","message":"s/and run/before running/\n\n?","commit_id":"c35c45802d3634875eed6322a2cf3b19413694ac"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"e892ce3188dcbb4e11e5711878159040f58973b9","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"--------------------------"},{"line_number":7,"context_line":"Transition to production"},{"line_number":8,"context_line":"--------------------------"},{"line_number":9,"context_line":"The erasure code support in Swift is considered \"ready for production\".  The"},{"line_number":10,"context_line":"major functionality is included, but we need to continue to test or validate"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_ad06f21a","line":7,"updated":"2015-12-18 15:56:24.000000000","message":"sorry, this really doesn\u0027t agree with me \"transition to production\".  I think the wording below is OK but this title should be changed to \"Not to Deployer\" or something as we don\u0027t need to invent some new stage between beta and production (or imply that we are)","commit_id":"2ce5ba94339a2ac77b126069379b92a3465e02eb"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"280871017b4c020a5ae48a5e55e061dc67816e91","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"--------------------------"},{"line_number":7,"context_line":"Transition to production"},{"line_number":8,"context_line":"--------------------------"},{"line_number":9,"context_line":"The erasure code support in Swift is considered \"ready for production\".  The"},{"line_number":10,"context_line":"major functionality is included, but we need to continue to test or validate"}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_4dcf4e29","line":7,"in_reply_to":"3a7e1126_ad06f21a","updated":"2015-12-18 16:10:57.000000000","message":"agree with Paul","commit_id":"2ce5ba94339a2ac77b126069379b92a3465e02eb"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"280871017b4c020a5ae48a5e55e061dc67816e91","unresolved":false,"context_lines":[{"line_number":6,"context_line":"--------------------------"},{"line_number":7,"context_line":"Transition to production"},{"line_number":8,"context_line":"--------------------------"},{"line_number":9,"context_line":"The erasure code support in Swift is considered \"ready for production\".  The"},{"line_number":10,"context_line":"major functionality is included, but we need to continue to test or validate"},{"line_number":11,"context_line":"erasure code runs at large scale.  Deployers are urged to do extensive testing"},{"line_number":12,"context_line":"before running production data using an erasure code storage policy."}],"source_content_type":"text/x-rst","patch_set":3,"id":"3a7e1126_8d1a167c","line":9,"updated":"2015-12-18 16:10:57.000000000","message":"This paragraph seems contradictory to me. The claim \"ready for production\" is followed by  (in my words) \"don\u0027t use in production until you have done lots of testing\".\n\nI understand that the intent is (probably) to convey that the status has moved on from \"don\u0027t use in production\" but I think it is sufficient to simply remove the beta/experimental label so that the EC feature becomes no different from the rest of the codebase.\n\nThe *changelog\" can record that the status has changed (i.e. a \"transition\" has occurred), the docs should simply say what it is.","commit_id":"2ce5ba94339a2ac77b126069379b92a3465e02eb"}],"etc/object-server.conf-sample":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"587f09918ff78e84d35815db32dfaf39294f68d1","unresolved":false,"context_lines":[{"line_number":172,"context_line":"# concurrency \u003d 1"},{"line_number":173,"context_line":"# stats_interval \u003d 300"},{"line_number":174,"context_line":"#"},{"line_number":175,"context_line":"# The sync method to use; default is rsync"},{"line_number":176,"context_line":"# sync_method \u003d rsync"},{"line_number":177,"context_line":"#"},{"line_number":178,"context_line":"# max duration of a partition rsync"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"3a7e1126_f71b6071","line":175,"updated":"2015-12-18 17:39:02.000000000","message":"maybe add\n\n  default is rsync, alternative is ssync\n\njust as a headsup that there is an alternative","commit_id":"551fe1177f21566f660d9baaa72aff2b69a3a25e"}]}