)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"6326174878103c01e96086d6acc51faec0e462ba","unresolved":false,"context_lines":[{"line_number":55,"context_line":"[2] http://wsgi-intercept.readthedocs.org/en/latest/"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"Change-Id: I0cbd40105b1f07a9775a9fc42bde50c233f0b950"},{"line_number":58,"context_line":"Related-Bug: #1337761"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"baada198_cd6d2d9b","line":58,"updated":"2014-07-25 10:38:19.000000000","message":"Just a nit: good practice is to have \"Change-Id:\" the last line in commit message.","commit_id":"1672407fd179d9d93963f8c06afbd8a6b5e566b2"}],"ceilometer/objectstore/swift_middleware.py":[{"author":{"_account_id":10987,"name":"Igor Degtiarov","username":"tovstun","inactive":true},"change_message_id":"6326174878103c01e96086d6acc51faec0e462ba","unresolved":false,"context_lines":[{"line_number":126,"context_line":"        try:"},{"line_number":127,"context_line":"            self._publish_sample(env, bytes_received, bytes_sent)"},{"line_number":128,"context_line":"        except Exception:"},{"line_number":129,"context_line":"            self.logger.exception(\u0027Failed to publish samples\u0027)"},{"line_number":130,"context_line":""},{"line_number":131,"context_line":"    def _publish_sample(self, env, bytes_received, bytes_sent):"},{"line_number":132,"context_line":"        req \u003d REQUEST.Request(env)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_980c5195","line":129,"updated":"2014-07-25 10:38:19.000000000","message":"Probably it would be better to log some info about exception, for example:\nexcept Exception as ex:\n    self.logger.exception(\u0027Failed to publish samples. Exception: %s\u0027 % ex)\n\nWhat do you think?","commit_id":"1672407fd179d9d93963f8c06afbd8a6b5e566b2"}]}