)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"change_message_id":"126bbeb369db88a71f22c86ae76563ee243bd21d","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- Fix a \u0027raise Exception class\u0027 syntax to \u0027raise Exception instance\u0027"},{"line_number":14,"context_line":"- Use original eventlet.Timeout instead of swift.exceptions.Timeout"},{"line_number":15,"context_line":"  imported from eventlet.Timout"},{"line_number":16,"context_line":"- Change Timeout to initiate w/o args (1st arguments should be timeout"},{"line_number":17,"context_line":"  second and we don\u0027t have to set it if we don\u0027t want to set the sec)"},{"line_number":18,"context_line":"- Add a message argument to some Exception instances"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_8a53a838","line":15,"updated":"2015-11-11 19:00:33.000000000","message":"nit: eventlet.Timeout","commit_id":"ef1f8ec365a0ea67cbd678e9fbb3efb0cd2e96de"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"8579d659e33dda4b13010279a1154332ed2a6d28","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"- Fix a \u0027raise Exception class\u0027 syntax to \u0027raise Exception instance\u0027"},{"line_number":14,"context_line":"- Use original eventlet.Timeout instead of swift.exceptions.Timeout"},{"line_number":15,"context_line":"  imported from eventlet.Timout"},{"line_number":16,"context_line":"- Change Timeout to initiate w/o args (1st arguments should be timeout"},{"line_number":17,"context_line":"  second and we don\u0027t have to set it if we don\u0027t want to set the sec)"},{"line_number":18,"context_line":"- Add a message argument to some Exception instances"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"da85f559_59403b07","line":15,"in_reply_to":"da85f559_8a53a838","updated":"2015-11-11 21:50:17.000000000","message":"will fix, thanks!","commit_id":"ef1f8ec365a0ea67cbd678e9fbb3efb0cd2e96de"}],"test/unit/proxy/controllers/test_obj.py":[{"author":{"_account_id":15932,"name":"Bill Huber","email":"wbhuber@us.ibm.com","username":"wbhuber"},"change_message_id":"ee1ee8a5fb6aa6036940034e2ae6f1f7b6abc38e","unresolved":false,"context_lines":[{"line_number":613,"context_line":"            self.app.error_suppression_limit + 1)"},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"    def test_PUT_error_during_transfer_data(self):"},{"line_number":616,"context_line":"        class FakeReader(object):"},{"line_number":617,"context_line":"            def read(self, size):"},{"line_number":618,"context_line":"                raise exceptions.ChunkReadError(None)"},{"line_number":619,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_7189a674","line":616,"updated":"2015-11-11 16:28:32.000000000","message":"This exception doesn\u0027t have a exception message whereas in test_PUT_ec_error_during_transfer_data() has a message (for EC section).  Why is this different from that?","commit_id":"ef1f8ec365a0ea67cbd678e9fbb3efb0cd2e96de"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"8579d659e33dda4b13010279a1154332ed2a6d28","unresolved":false,"context_lines":[{"line_number":613,"context_line":"            self.app.error_suppression_limit + 1)"},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"    def test_PUT_error_during_transfer_data(self):"},{"line_number":616,"context_line":"        class FakeReader(object):"},{"line_number":617,"context_line":"            def read(self, size):"},{"line_number":618,"context_line":"                raise exceptions.ChunkReadError(None)"},{"line_number":619,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_39c9b759","line":616,"in_reply_to":"da85f559_3a7693a4","updated":"2015-11-11 21:50:17.000000000","message":"exactly, the message should be consistent. Nice catch, thanks.\n\nWill fix.","commit_id":"ef1f8ec365a0ea67cbd678e9fbb3efb0cd2e96de"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"859a9c7a860339717e77b540e24c86e6ebe11a14","unresolved":false,"context_lines":[{"line_number":613,"context_line":"            self.app.error_suppression_limit + 1)"},{"line_number":614,"context_line":""},{"line_number":615,"context_line":"    def test_PUT_error_during_transfer_data(self):"},{"line_number":616,"context_line":"        class FakeReader(object):"},{"line_number":617,"context_line":"            def read(self, size):"},{"line_number":618,"context_line":"                raise exceptions.ChunkReadError(None)"},{"line_number":619,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_3a7693a4","line":616,"in_reply_to":"da85f559_7189a674","updated":"2015-11-11 17:11:08.000000000","message":"Question for either of you, what\u0027s the purpose of adding the generic \u0027exception message\u0027 string anyway?  If it does something cool I\u0027d think you\u0027d want to be consistent and do it here too :)","commit_id":"ef1f8ec365a0ea67cbd678e9fbb3efb0cd2e96de"},{"author":{"_account_id":2622,"name":"Samuel Merritt","email":"spam+launchpad@andcheese.org","username":"torgomatic"},"change_message_id":"126bbeb369db88a71f22c86ae76563ee243bd21d","unresolved":false,"context_lines":[{"line_number":630,"context_line":"    def test_PUT_chunkreadtimeout_during_transfer_data(self):"},{"line_number":631,"context_line":"        class FakeReader(object):"},{"line_number":632,"context_line":"            def read(self, size):"},{"line_number":633,"context_line":"                raise exceptions.ChunkReadTimeout()"},{"line_number":634,"context_line":""},{"line_number":635,"context_line":"        req \u003d swob.Request.blank(\u0027/v1/a/c/o.jpg\u0027, method\u003d\u0027PUT\u0027,"},{"line_number":636,"context_line":"                                 body\u003d\u0027test body\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_2a339c15","line":633,"updated":"2015-11-11 19:00:33.000000000","message":"Okay. I thought this was a dangerous change, but I looked at the eventlet code and None is actually the default here.","commit_id":"ef1f8ec365a0ea67cbd678e9fbb3efb0cd2e96de"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"8579d659e33dda4b13010279a1154332ed2a6d28","unresolved":false,"context_lines":[{"line_number":630,"context_line":"    def test_PUT_chunkreadtimeout_during_transfer_data(self):"},{"line_number":631,"context_line":"        class FakeReader(object):"},{"line_number":632,"context_line":"            def read(self, size):"},{"line_number":633,"context_line":"                raise exceptions.ChunkReadTimeout()"},{"line_number":634,"context_line":""},{"line_number":635,"context_line":"        req \u003d swob.Request.blank(\u0027/v1/a/c/o.jpg\u0027, method\u003d\u0027PUT\u0027,"},{"line_number":636,"context_line":"                                 body\u003d\u0027test body\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"da85f559_ffc22f52","line":633,"in_reply_to":"da85f559_2a339c15","updated":"2015-11-11 21:50:17.000000000","message":"yes. ChunkReadTimeout is a child class of eventlet.Timeout and the first argument of eventlet.Timeout.__init__() is None in default.","commit_id":"ef1f8ec365a0ea67cbd678e9fbb3efb0cd2e96de"}]}