)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5a41fdd4de3ab3c6858e0b7400df7a0408a89026","unresolved":false,"context_lines":[{"line_number":11,"context_line":"foo, default). This is an incorrect change, because the default of"},{"line_number":12,"context_line":"getattr() is only used if AttributeError is raised. The original get()"},{"line_number":13,"context_line":"call would return the default if the variable was not set, but in order"},{"line_number":14,"context_line":"to get the default with getattr() \u0027in\u0027 has to be used. So now, the"},{"line_number":15,"context_line":"getattr() calls are changed to:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"foo \u003d aggregate.foo if \u0027foo\u0027 in aggregate else default"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Also, a comment was added for using getattr() on all aggregate fields,"},{"line_number":20,"context_line":"explaining we can do the getattr() and not worry about metadata being"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5aef4532_4269dd24","line":17,"range":{"start_line":14,"start_character":55,"end_line":17,"end_character":54},"updated":"2016-03-08 16:36:29.000000000","message":"Where does this happen since I don\u0027t see it happening in this change?  All I see happening here is documentation and tests.","commit_id":"d13804318d9316b3f4f1627ed6b967e7aab0e1b6"}]}