)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"406e8396595dc1565851236dc2d89635995a419e","unresolved":false,"context_lines":[{"line_number":9,"context_line":"In file:"},{"line_number":10,"context_line":"  trove/guestagent/strategies/restore/experimental/couchbase_impl.py:195"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    LOG.error(_(\"cbrestore failed. Retrying...\"))"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This was replaced by:"},{"line_number":15,"context_line":"    LOG.exception(_(\"cbrestore failed. Retrying...\"))"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a5de9d1_e866cbeb","line":12,"updated":"2016-02-01 18:06:54.000000000","message":"You should amend this ;)","commit_id":"fe823e22d3e92618f2acb17fce19db09a555b889"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"30f7fa07eb17004a455447ea39e3bcaf46112489","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    LOG.error(_(\"cbrestore failed. Retrying...\"))"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"This was replaced by:"},{"line_number":15,"context_line":"    LOG.exception(_(\"cbrestore failed. Retrying...\"))"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ifb37a934b317febe992a2bce8d94b694fc2e1b8a"},{"line_number":18,"context_line":"Closes-Bug: #1537478"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"fa0719c6_5e2a3e7f","line":15,"updated":"2016-03-22 17:37:49.000000000","message":"this should be the change. but you did more in the code.","commit_id":"fe823e22d3e92618f2acb17fce19db09a555b889"}],"trove/guestagent/strategies/restore/experimental/couchbase_impl.py":[{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"8db299bf086dde9559e6141b5e6dc6f1e41f760e","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                        # http://www.couchbase.com/issues/browse/MB-10832"},{"line_number":193,"context_line":"                        # Retrying typically works"},{"line_number":194,"context_line":"                        LOG.error(p)"},{"line_number":195,"context_line":"                        LOG.exception(_(\"cbrestore failed. Retrying...\"))"},{"line_number":196,"context_line":"                        utils.execute_with_timeout(restore_cmd,"},{"line_number":197,"context_line":"                                                   shell\u003dTrue,"},{"line_number":198,"context_line":"                                                   timeout\u003d300)"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a68dd71_5f5c747c","line":195,"updated":"2016-01-25 15:08:37.000000000","message":"The way you\u0027d normally do this is to merge it with the previous LOG.error line.\n\nLOG.exception(... % str(p))","commit_id":"1e378345952e41bc462f26a42bf18e85f3fcf6dc"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"464c4196424526f403c1ba6c63da6526c79487af","unresolved":false,"context_lines":[{"line_number":187,"context_line":"                        utils.execute_with_timeout(restore_cmd,"},{"line_number":188,"context_line":"                                                   shell\u003dTrue,"},{"line_number":189,"context_line":"                                                   timeout\u003d300)"},{"line_number":190,"context_line":"                    except exception.ProcessExecutionError as p:"},{"line_number":191,"context_line":"                        # cbrestore fails or hangs at times:"},{"line_number":192,"context_line":"                        # http://www.couchbase.com/issues/browse/MB-10832"},{"line_number":193,"context_line":"                        # Retrying typically works"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa0719c6_ec46d379","line":190,"range":{"start_line":190,"start_character":58,"end_line":190,"end_character":63},"updated":"2016-03-23 15:40:22.000000000","message":"This can also be removed, as it won\u0027t be necessary","commit_id":"fe823e22d3e92618f2acb17fce19db09a555b889"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"90556256925156c35611ab8ce24941f2161eda78","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                        # http://www.couchbase.com/issues/browse/MB-10832"},{"line_number":193,"context_line":"                        # Retrying typically works"},{"line_number":194,"context_line":"                        LOG.exception(_(\"%s cbrestore failed. Retrying...\"),"},{"line_number":195,"context_line":"                                      str(p))"},{"line_number":196,"context_line":"                        utils.execute_with_timeout(restore_cmd,"},{"line_number":197,"context_line":"                                                   shell\u003dTrue,"},{"line_number":198,"context_line":"                                                   timeout\u003d300)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa0719c6_fb26c4cc","line":195,"updated":"2016-03-22 17:36:12.000000000","message":"You shouldn\u0027t need to cast the exception to a string if you use the LOG.exception method. It prints the exception stack automatically. (this is the much preferred way of printing out the stack trace of an exception rather than LOG.error or any other way.)","commit_id":"fe823e22d3e92618f2acb17fce19db09a555b889"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"464c4196424526f403c1ba6c63da6526c79487af","unresolved":false,"context_lines":[{"line_number":192,"context_line":"                        # http://www.couchbase.com/issues/browse/MB-10832"},{"line_number":193,"context_line":"                        # Retrying typically works"},{"line_number":194,"context_line":"                        LOG.exception(_(\"%s cbrestore failed. Retrying...\"),"},{"line_number":195,"context_line":"                                      str(p))"},{"line_number":196,"context_line":"                        utils.execute_with_timeout(restore_cmd,"},{"line_number":197,"context_line":"                                                   shell\u003dTrue,"},{"line_number":198,"context_line":"                                                   timeout\u003d300)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa0719c6_ac9b3b17","line":195,"in_reply_to":"fa0719c6_fb26c4cc","updated":"2016-03-23 15:40:22.000000000","message":"I agree - the commit message has the right code","commit_id":"fe823e22d3e92618f2acb17fce19db09a555b889"},{"author":{"_account_id":9782,"name":"Morgan Jones","email":"morgan@tesora.com","username":"vgnbkr"},"change_message_id":"a32a18eb6165b878731348b5d9b17bc011100ac7","unresolved":false,"context_lines":[{"line_number":191,"context_line":"                        # cbrestore fails or hangs at times:"},{"line_number":192,"context_line":"                        # http://www.couchbase.com/issues/browse/MB-10832"},{"line_number":193,"context_line":"                        # Retrying typically works"},{"line_number":194,"context_line":"                        LOG.exception(_(\"%s cbrestore failed. Retrying...\"))"},{"line_number":195,"context_line":"                        utils.execute_with_timeout(restore_cmd,"},{"line_number":196,"context_line":"                                                   shell\u003dTrue,"},{"line_number":197,"context_line":"                                                   timeout\u003d300)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a122d0e_0b8c517e","line":194,"updated":"2016-05-02 18:41:02.000000000","message":"Don\u0027t think you need the %s here.","commit_id":"7aa0089eb892dba7fe0daa547f83cabd528931ff"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"fea6b973334b82e1567f5a46547f13f28401a372","unresolved":false,"context_lines":[{"line_number":191,"context_line":"                        # cbrestore fails or hangs at times:"},{"line_number":192,"context_line":"                        # http://www.couchbase.com/issues/browse/MB-10832"},{"line_number":193,"context_line":"                        # Retrying typically works"},{"line_number":194,"context_line":"                        LOG.exception(_(\"%s cbrestore failed. Retrying...\"))"},{"line_number":195,"context_line":"                        utils.execute_with_timeout(restore_cmd,"},{"line_number":196,"context_line":"                                                   shell\u003dTrue,"},{"line_number":197,"context_line":"                                                   timeout\u003d300)"}],"source_content_type":"text/x-python","patch_set":4,"id":"1a122d0e_ebf6358f","line":194,"in_reply_to":"1a122d0e_0b8c517e","updated":"2016-05-02 18:49:24.000000000","message":"Really? \n\nThanks Morgan, you are right. Will fix","commit_id":"7aa0089eb892dba7fe0daa547f83cabd528931ff"}]}