)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"00ad8316337a919403475e3daa1a0d4bb3cd29d1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     nicky \u003cpomelonicky@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-11-30 10:35:07 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a configuration option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"add rpc_max_timeout option to _client_opts"},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_d47326b2","line":7,"updated":"2018-12-07 14:20:56.000000000","message":"When this shows up in git log output it\u0027s not going to be especially helpful for understanding what is in the commit. Could you include the name of the option?","commit_id":"993c9bb824da13757f85c791879b94b0415f51f3"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"00ad8316337a919403475e3daa1a0d4bb3cd29d1","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add a configuration option"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"add rpc_max_timeout option to _client_opts"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I816be345c90b6a4e0a501a56e672030a148041a8"},{"line_number":12,"context_line":"Related-Bug: #1805769"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_746492e8","line":9,"updated":"2018-12-07 14:20:56.000000000","message":"This is saying what the patch does, but not why. Could you explain what benefit the patch has?","commit_id":"993c9bb824da13757f85c791879b94b0415f51f3"}],"oslo_messaging/rpc/client.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"00ad8316337a919403475e3daa1a0d4bb3cd29d1","unresolved":false,"context_lines":[{"line_number":44,"context_line":"               help\u003d\u0027Seconds to wait for a response from a call.\u0027),"},{"line_number":45,"context_line":"    cfg.IntOpt(\u0027rpc_max_timeout\u0027,"},{"line_number":46,"context_line":"               default\u003d100,"},{"line_number":47,"context_line":"               help\u003d\u0027max seconds to wait for a response from a call.\u0027),"},{"line_number":48,"context_line":"]"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_b4930ab0","line":47,"updated":"2018-12-07 14:20:56.000000000","message":"Nothing seems to use this option. Is the patch complete?","commit_id":"993c9bb824da13757f85c791879b94b0415f51f3"}]}