)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"e1fc073db6bb5490379cbc09114b945e5cfbb9e6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Cao ShuFeng \u003ccaosf.fnst@cn.fujitsu.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-04-08 13:32:42 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ehanche error message [1/5]"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change is intend to implement this approved guideline[1]."},{"line_number":10,"context_line":"It provides an example of this spec[2] which is still in review."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5a18252c_e783fcca","line":7,"updated":"2016-04-12 17:23:11.000000000","message":"\"enhance\"","commit_id":"ad94d4d35fb92ccc9930d83791d8d51e1b5ede00"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"e1fc073db6bb5490379cbc09114b945e5cfbb9e6","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-04-08 13:32:42 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"ehanche error message [1/5]"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This change is intend to implement this approved guideline[1]."},{"line_number":10,"context_line":"It provides an example of this spec[2] which is still in review."},{"line_number":11,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"5a18252c_6721acfb","line":8,"updated":"2016-04-12 17:23:11.000000000","message":"Please add more info here about what this change actually does.  (Rather than just linking to a spec.)","commit_id":"ad94d4d35fb92ccc9930d83791d8d51e1b5ede00"}],"cinder/exception_meta.py":[{"author":{"_account_id":11903,"name":"Anthony Lee","email":"anthony.mic.lee@hpe.com","username":"leeantho"},"change_message_id":"ff1953d2cbac3d393b8bc166061b29d1588f2803","unresolved":false,"context_lines":[{"line_number":33,"context_line":"            \u0027title\u0027: self.title or self.TITLE,"},{"line_number":34,"context_line":"            \u0027detail\u0027: self.detail or self.DETAIL,"},{"line_number":35,"context_line":"            \u0027links\u0027: self.links or self.LINKS}"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"class DefaultExceptionMeta(BaseExceptionMeta):"},{"line_number":39,"context_line":"    CODE \u003d \u0027volume.api.undefined-code\u0027"}],"source_content_type":"text/x-python","patch_set":6,"id":"1a122d0e_397963c9","line":36,"updated":"2016-04-18 21:25:34.000000000","message":"Why not have all the child classes of BaseExceptionMeta use the already defined variables for status, title, etc?\n\nIt would eliminate the need for the or statements above \"self.status or self.STATUS\", \"self.title or self.TITLE\", etc.\n\nAlso self.CODE does not exist for the BaseExceptionMeta class so if an instance of BaseExceptionMeta had its to_dict method called it would fail.","commit_id":"18f9638b99b439453de741e8b4c53c487a24fc40"}]}