)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7536,"name":"Ilya Pekelny","email":"ipekelny@mirantis.com","username":"I159"},"change_message_id":"8e346d7e7d65d761f04173ccb10884c09149ad6b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove noop code"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"These lines of code didn\u0027t do anything."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ie03a9164a5f4e9b1aa5622eb44bbe891015a26be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F8p0s%3D","line":9,"updated":"2013-12-23 16:58:52.000000000","message":"Explain please, why this code does not do anything?","commit_id":"0f584a2daaa3ff4d16c499c4712ef7b693a0b7bd"},{"author":{"_account_id":6460,"name":"Brad Topol","email":"btopol@us.ibm.com","username":"btopol"},"change_message_id":"be84bad4b204103133902836a94a3431b769b069","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove noop code"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"These lines of code didn\u0027t do anything."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ie03a9164a5f4e9b1aa5622eb44bbe891015a26be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F8mXM%3D","line":9,"updated":"2013-12-24 14:50:34.000000000","message":"I agree with Ilya.  If you could give a little more explanation it would be helpful.  Other than that looks great","commit_id":"0f584a2daaa3ff4d16c499c4712ef7b693a0b7bd"},{"author":{"_account_id":8996,"name":"Florent Flament","email":"contact@florentflament.com","username":"florent"},"change_message_id":"eb4e2da4efddd4c1b0352eb01cf679224d49d667","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Remove noop code"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"These lines of code didn\u0027t do anything."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: Ie03a9164a5f4e9b1aa5622eb44bbe891015a26be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAT3%2F%2F57A%3D","line":9,"in_reply_to":"AAAATn%2F8mXM%3D","updated":"2013-12-27 12:41:11.000000000","message":"I agree with Brant. The only purpose of logging.getLogger is to return a logger, that can be used afterwards. If the return value is not used, the call doesn\u0027t do anything:\nhttp://docs.python.org/2/library/logging.html#module-level-functions","commit_id":"0f584a2daaa3ff4d16c499c4712ef7b693a0b7bd"},{"author":{"_account_id":8996,"name":"Florent Flament","email":"contact@florentflament.com","username":"florent"},"change_message_id":"13c4852c4ad808da4e1c41456b0ad6d821990655","unresolved":false,"context_lines":[{"line_number":12,"context_line":"The comment says that this code was supposed to change the logging"},{"line_number":13,"context_line":"configuration but it doesn\u0027t do that. Also, this isn\u0027t the correct"},{"line_number":14,"context_line":"way to change the logging configuration, see"},{"line_number":15,"context_line":"https://review.openstack.org/#/c/62068/."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ie03a9164a5f4e9b1aa5622eb44bbe891015a26be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2Fs0g%3D","line":15,"updated":"2014-01-03 08:31:19.000000000","message":"You should remove the \u0027.\u0027 at the end of line. It makes the link broken.","commit_id":"3cfe95b7fed061106264c4416aff921d28bac285"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"1eaf68cdf15e0d82e641088fba4f44b0a182f6d3","unresolved":false,"context_lines":[{"line_number":12,"context_line":"The comment says that this code was supposed to change the logging"},{"line_number":13,"context_line":"configuration but it doesn\u0027t do that. Also, this isn\u0027t the correct"},{"line_number":14,"context_line":"way to change the logging configuration, see"},{"line_number":15,"context_line":"https://review.openstack.org/#/c/62068/."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: Ie03a9164a5f4e9b1aa5622eb44bbe891015a26be"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAT3%2F%2Fr1Q%3D","line":15,"in_reply_to":"AAAAT3%2F%2Fs0g%3D","updated":"2014-01-03 14:35:54.000000000","message":"Done","commit_id":"3cfe95b7fed061106264c4416aff921d28bac285"}]}