)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":19134,"name":"Eyal","email":"eyalb1@gmail.com","username":"eyalb"},"change_message_id":"0e5824ec6711b4e4b8b93aafc3e089bc624717ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3f4b6375_5c1b8172","updated":"2017-10-30 10:09:04.000000000","message":"could you please describe exactly what you fixed in the comment","commit_id":"1e865c12cdbb307b1be0d5bfe351b1f941d12433"},{"author":{"_account_id":18783,"name":"dongwenjuan","email":"dong.wenjuan@zte.com.cn","username":"dongwenjuan"},"change_message_id":"101a7c756b3578d4a566d4d66e69fcd43c7d6e32","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3f4b6375_3cbb250f","in_reply_to":"3f4b6375_5c1b8172","updated":"2017-10-30 10:21:09.000000000","message":"Done","commit_id":"1e865c12cdbb307b1be0d5bfe351b1f941d12433"}],"vitrage/datasources/aodh/driver.py":[{"author":{"_account_id":19853,"name":"Nguyen Hung Phuong","email":"phuongnh@vn.fujitsu.com","username":"phuongnh"},"change_message_id":"bae15c90a2216b70069f54dc9e8498281065151e","unresolved":false,"context_lines":[{"line_number":287,"context_line":"        elif entity_detail[AodhProps.TYPE] \u003d\u003d \\"},{"line_number":288,"context_line":"                AodhProps.GNOCCHI_RESOURCES_THRESHOLD:"},{"line_number":289,"context_line":"            entity_detail.update("},{"line_number":290,"context_line":"                cls._convert_gnocchi_resources_threshold_alarm_event(alarm_rule))"},{"line_number":291,"context_line":""},{"line_number":292,"context_line":"        return entity_detail"},{"line_number":293,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f4b6375_e745ff44","line":290,"range":{"start_line":290,"start_character":16,"end_line":290,"end_character":81},"updated":"2017-10-30 03:54:22.000000000","message":"Please fix pep8 issue: line too long","commit_id":"4b1ca3a5d73ec1dfd9dbb257daf14266fb061923"},{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"d10e47aea413bf150be4f8a660aecafbc7bc95c5","unresolved":false,"context_lines":[{"line_number":242,"context_line":"        }"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    @classmethod"},{"line_number":245,"context_line":"    def _convert_threshold_alarm_event(cls, rule):"},{"line_number":246,"context_line":"        return {"},{"line_number":247,"context_line":"            AodhProps.RESOURCE_ID: _parse_query(rule, AodhProps.RESOURCE_ID)"},{"line_number":248,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_0b838558","line":245,"range":{"start_line":245,"start_character":44,"end_line":245,"end_character":48},"updated":"2017-10-31 07:52:55.000000000","message":"Why changing event to rule? I think the method name is convert \"event\"","commit_id":"ab0e7a719a85170de48d3f750e354ea4e2c0c5a8"},{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"d10e47aea413bf150be4f8a660aecafbc7bc95c5","unresolved":false,"context_lines":[{"line_number":242,"context_line":"        }"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    @classmethod"},{"line_number":245,"context_line":"    def _convert_threshold_alarm_event(cls, rule):"},{"line_number":246,"context_line":"        return {"},{"line_number":247,"context_line":"            AodhProps.RESOURCE_ID: _parse_query(rule, AodhProps.RESOURCE_ID)"},{"line_number":248,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_55dfb639","line":245,"range":{"start_line":245,"start_character":44,"end_line":245,"end_character":48},"updated":"2017-10-31 07:52:55.000000000","message":"I think `rule` is just one property of the event. If we are parsing the event, name it as event.","commit_id":"a3f5523068591dfe18b8708683e0e0e0fade9dcd"},{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"fce87723077362199c1e5b0a57a9f21c544dd9ca","unresolved":false,"context_lines":[{"line_number":242,"context_line":"        }"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    @classmethod"},{"line_number":245,"context_line":"    def _convert_threshold_alarm_event(cls, rule):"},{"line_number":246,"context_line":"        return {"},{"line_number":247,"context_line":"            AodhProps.RESOURCE_ID: _parse_query(rule, AodhProps.RESOURCE_ID)"},{"line_number":248,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_c075269a","line":245,"range":{"start_line":245,"start_character":44,"end_line":245,"end_character":48},"in_reply_to":"3f4b6375_5537b64e","updated":"2017-10-31 08:27:16.000000000","message":"Renaming the method makes sense :-)","commit_id":"a3f5523068591dfe18b8708683e0e0e0fade9dcd"},{"author":{"_account_id":18783,"name":"dongwenjuan","email":"dong.wenjuan@zte.com.cn","username":"dongwenjuan"},"change_message_id":"cc46260784c81387cc933206437bd3e835260090","unresolved":false,"context_lines":[{"line_number":242,"context_line":"        }"},{"line_number":243,"context_line":""},{"line_number":244,"context_line":"    @classmethod"},{"line_number":245,"context_line":"    def _convert_threshold_alarm_event(cls, rule):"},{"line_number":246,"context_line":"        return {"},{"line_number":247,"context_line":"            AodhProps.RESOURCE_ID: _parse_query(rule, AodhProps.RESOURCE_ID)"},{"line_number":248,"context_line":"        }"}],"source_content_type":"text/x-python","patch_set":4,"id":"3f4b6375_5537b64e","line":245,"range":{"start_line":245,"start_character":44,"end_line":245,"end_character":48},"in_reply_to":"3f4b6375_55dfb639","updated":"2017-10-31 08:04:40.000000000","message":"Here only parse the rule to get the neighbor of the entity.\nI rename the function as \u0027_convert_threshold_alarm_rule\u0027 in the patch of rewrite Aodh datasource.","commit_id":"a3f5523068591dfe18b8708683e0e0e0fade9dcd"}]}