)]}'
{"vitrage/datasources/alarm_transformer_base.py":[{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"2f0b9d80993ecc889deff198e7e1eb52dc5091f3","unresolved":false,"context_lines":[{"line_number":59,"context_line":""},{"line_number":60,"context_line":"    def _get_alarm_state(self, entity_event):"},{"line_number":61,"context_line":"        event_type \u003d entity_event.get(DSProps.EVENT_TYPE, None)"},{"line_number":62,"context_line":"        if event_type:"},{"line_number":63,"context_line":"            return AlarmProps.ALARM_INACTIVE_STATE if \\"},{"line_number":64,"context_line":"                EventAction.DELETE_ENTITY \u003d\u003d event_type else \\"},{"line_number":65,"context_line":"                AlarmProps.ALARM_ACTIVE_STATE"}],"source_content_type":"text/x-python","patch_set":2,"id":"7aa08908_584ab984","line":62,"updated":"2016-06-06 08:57:27.000000000","message":"For readability, please consider using explicit test of None value.\n\nSee http://stackoverflow.com/questions/3965104/not-none-test-in-python","commit_id":"6cc6c387dfc6b918c0ce7d90f8698c9ac7838ded"}],"vitrage/datasources/aodh/transformer.py":[{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"4a5c5e8ac1244e272f930138b5b50a096104b99a","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        event_type \u003d entity_event.get(DSProps.EVENT_TYPE, None)"},{"line_number":73,"context_line":"        entity_state \u003d AlarmProps.ALARM_INACTIVE_STATE if \\"},{"line_number":74,"context_line":"            EventAction.DELETE_ENTITY \u003d\u003d event_type else \\"},{"line_number":75,"context_line":"            AlarmProps.ALARM_ACTIVE_STATE"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        return graph_utils.create_vertex("},{"line_number":78,"context_line":"            self._create_entity_key(entity_event),"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_962b5894","line":75,"range":{"start_line":75,"start_character":23,"end_line":75,"end_character":41},"updated":"2016-06-02 14:19:58.000000000","message":"I think we should have a common behavior for nagios and aodh, in alarm_transformer_base.py.","commit_id":"0dbbc78b69033a621ab8c0c01b3ce0552c4ca2ce"},{"author":{"_account_id":18783,"name":"dongwenjuan","email":"dong.wenjuan@zte.com.cn","username":"dongwenjuan"},"change_message_id":"c3c2e8f3bb87d7d4c5bb9ee9402d4c346924ffef","unresolved":false,"context_lines":[{"line_number":72,"context_line":"        event_type \u003d entity_event.get(DSProps.EVENT_TYPE, None)"},{"line_number":73,"context_line":"        entity_state \u003d AlarmProps.ALARM_INACTIVE_STATE if \\"},{"line_number":74,"context_line":"            EventAction.DELETE_ENTITY \u003d\u003d event_type else \\"},{"line_number":75,"context_line":"            AlarmProps.ALARM_ACTIVE_STATE"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"        return graph_utils.create_vertex("},{"line_number":78,"context_line":"            self._create_entity_key(entity_event),"}],"source_content_type":"text/x-python","patch_set":1,"id":"9abb7d3a_dcf5decb","line":75,"range":{"start_line":75,"start_character":23,"end_line":75,"end_character":41},"in_reply_to":"9abb7d3a_962b5894","updated":"2016-06-03 01:54:18.000000000","message":"OK，i\u0027ll fix it,thanks.","commit_id":"0dbbc78b69033a621ab8c0c01b3ce0552c4ca2ce"}],"vitrage/tests/unit/datasources/nagios/test_nagios_transformer.py":[{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"fa039a87c9a2eb3c01f9af1b41d65f3e034575d2","unresolved":false,"context_lines":[{"line_number":109,"context_line":""},{"line_number":110,"context_line":"        event_type \u003d event.get(DSProps.EVENT_TYPE, None)"},{"line_number":111,"context_line":"        if event_type is not None:"},{"line_number":112,"context_line":"            event_status \u003d AlarmProps.ALARM_INACTIVE_STATE if \\"},{"line_number":113,"context_line":"                EventAction.DELETE_ENTITY \u003d\u003d event_type else \\"},{"line_number":114,"context_line":"                AlarmProps.ALARM_ACTIVE_STATE"},{"line_number":115,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_8ac92a7b","line":112,"range":{"start_line":112,"start_character":12,"end_line":112,"end_character":24},"updated":"2016-06-07 06:05:24.000000000","message":"I think the name event_status is confusing, since NagiosProperties.STATUS means something else.\nHow about you call it expected_event_state, or expected_state?","commit_id":"f2f61656c24f6b9f69fc419d6c24ad553b561b6f"},{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"fa039a87c9a2eb3c01f9af1b41d65f3e034575d2","unresolved":false,"context_lines":[{"line_number":118,"context_line":"                AlarmProps.ALARM_ACTIVE_STATE"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"        self.assertEqual(event_status,"},{"line_number":121,"context_line":"                         vertex[VProps.STATE])"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"        self.assertEqual(event[NagiosProperties.STATUS],"},{"line_number":124,"context_line":"                         vertex[VProps.SEVERITY])"}],"source_content_type":"text/x-python","patch_set":4,"id":"7aa08908_ca885225","line":121,"range":{"start_line":121,"start_character":0,"end_line":121,"end_character":25},"updated":"2016-06-07 06:05:24.000000000","message":"You can write it in one line","commit_id":"f2f61656c24f6b9f69fc419d6c24ad553b561b6f"}]}