)]}'
{"vitrage/entity_graph/consistency/service.py":[{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"487e9cc8f9645cb230b6ee8cfe2f3d6ba21bd0a5","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        super(VitrageConsistencyService, self).start()"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"        consistency_enf \u003d ConsistencyEnforcer(self.conf,"},{"line_number":43,"context_line":"                                              self.actions_notifier.notify,"},{"line_number":44,"context_line":"                                              self.entity_graph)"},{"line_number":45,"context_line":"        self.tg.add_timer(self.conf.datasources.snapshots_interval,"},{"line_number":46,"context_line":"                          consistency_enf.periodic_process,"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_f23ff2d3","line":43,"range":{"start_line":43,"start_character":46,"end_line":43,"end_character":74},"updated":"2017-12-04 09:13:53.000000000","message":"write that it\u0027s the actions_callback","commit_id":"87b24cb033d0ec7a27b64204c1d885ebc1e17804"}],"vitrage/entity_graph/service.py":[{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"487e9cc8f9645cb230b6ee8cfe2f3d6ba21bd0a5","unresolved":false,"context_lines":[{"line_number":70,"context_line":"LOW_DELAY \u003d 0.05"},{"line_number":71,"context_line":""},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"class TwoPriorityListener(object):"},{"line_number":74,"context_line":"    def __init__(self, conf, do_work_func, topic_low, topic_high):"},{"line_number":75,"context_line":"        self._do_work_func \u003d do_work_func"},{"line_number":76,"context_line":"        self._conf \u003d conf"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_b20aca23","line":73,"range":{"start_line":73,"start_character":6,"end_line":73,"end_character":25},"updated":"2017-12-04 09:13:53.000000000","message":"Should it be on the same file?","commit_id":"87b24cb033d0ec7a27b64204c1d885ebc1e17804"},{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"487e9cc8f9645cb230b6ee8cfe2f3d6ba21bd0a5","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    def _do_low_priority_work(self, event):"},{"line_number":104,"context_line":"        while True:"},{"line_number":105,"context_line":"            self._lock.acquire()"},{"line_number":106,"context_line":"            if not (time.time() - self._high_event_finish_time) \u003e LOW_DELAY:"},{"line_number":107,"context_line":"                self._lock.release()"},{"line_number":108,"context_line":"                time.sleep(LOW_DELAY)"},{"line_number":109,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_261c3b69","line":106,"range":{"start_line":106,"start_character":15,"end_line":106,"end_character":18},"updated":"2017-12-04 09:13:53.000000000","message":"Consider replacing it with a positive condition (if time \u003c DELAY)","commit_id":"87b24cb033d0ec7a27b64204c1d885ebc1e17804"},{"author":{"_account_id":19184,"name":"Idan Hefetz","email":"idan.hefetz@nokia.com","username":"idan-hefetz"},"change_message_id":"63ab11721398aa0aa53b034b372af7afb03202d5","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    def _do_low_priority_work(self, event):"},{"line_number":104,"context_line":"        while True:"},{"line_number":105,"context_line":"            self._lock.acquire()"},{"line_number":106,"context_line":"            if not (time.time() - self._high_event_finish_time) \u003e LOW_DELAY:"},{"line_number":107,"context_line":"                self._lock.release()"},{"line_number":108,"context_line":"                time.sleep(LOW_DELAY)"},{"line_number":109,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_50e33134","line":106,"range":{"start_line":106,"start_character":15,"end_line":106,"end_character":18},"in_reply_to":"df87a7cf_261c3b69","updated":"2017-12-04 14:25:39.000000000","message":"Done","commit_id":"87b24cb033d0ec7a27b64204c1d885ebc1e17804"},{"author":{"_account_id":19159,"name":"Ifat Afek","email":"ifat.afek@nokia.com","username":"ifat_afek"},"change_message_id":"487e9cc8f9645cb230b6ee8cfe2f3d6ba21bd0a5","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"class PushNotificationsEndpoint(object):"},{"line_number":122,"context_line":"    def __init__(self, func):"},{"line_number":123,"context_line":"        self.func \u003d func"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    def info(self, ctxt, publisher_id, event_type, payload, metadata):"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_06525714","line":122,"range":{"start_line":122,"start_character":23,"end_line":122,"end_character":27},"updated":"2017-12-04 09:13:53.000000000","message":"Why did you rename \u0027callback\u0027 to \u0027func\u0027? I think callback is more clear","commit_id":"87b24cb033d0ec7a27b64204c1d885ebc1e17804"},{"author":{"_account_id":19184,"name":"Idan Hefetz","email":"idan.hefetz@nokia.com","username":"idan-hefetz"},"change_message_id":"63ab11721398aa0aa53b034b372af7afb03202d5","unresolved":false,"context_lines":[{"line_number":119,"context_line":""},{"line_number":120,"context_line":""},{"line_number":121,"context_line":"class PushNotificationsEndpoint(object):"},{"line_number":122,"context_line":"    def __init__(self, func):"},{"line_number":123,"context_line":"        self.func \u003d func"},{"line_number":124,"context_line":""},{"line_number":125,"context_line":"    def info(self, ctxt, publisher_id, event_type, payload, metadata):"}],"source_content_type":"text/x-python","patch_set":1,"id":"df87a7cf_30e6ad46","line":122,"range":{"start_line":122,"start_character":23,"end_line":122,"end_character":27},"in_reply_to":"df87a7cf_06525714","updated":"2017-12-04 14:25:39.000000000","message":"Done","commit_id":"87b24cb033d0ec7a27b64204c1d885ebc1e17804"}],"vitrage/entity_graph/vitrage_init.py":[{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"99da5d56e1a560222096614966480ebd29c92214","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"            self.evaluator.run_evaluator()"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"            # if not self._wait_for_action(self.evaluator_queue.empty):"},{"line_number":49,"context_line":"            #     LOG.error(\u0027Evaluator Queue Not Empty\u0027)"},{"line_number":50,"context_line":"            #"},{"line_number":51,"context_line":"            # self._mark_old_deduced_alarms_as_deleted(timestamp, self.graph,"},{"line_number":52,"context_line":"            #                                          self.evaluator_queue)"},{"line_number":53,"context_line":"            self.status \u003d self.FINISHED"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"            LOG.info(\u0027Init Finished\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"df87a7cf_776d0197","line":52,"range":{"start_line":48,"start_character":0,"end_line":52,"end_character":76},"updated":"2017-12-04 20:22:11.000000000","message":"Is it commented out temporarily? For what reason?","commit_id":"4e71b45753447edaf51a0f328abf97ea2ca65582"},{"author":{"_account_id":19184,"name":"Idan Hefetz","email":"idan.hefetz@nokia.com","username":"idan-hefetz"},"change_message_id":"3568124a7cf631f47ca1711b27ed11887d0b2fb6","unresolved":false,"context_lines":[{"line_number":45,"context_line":""},{"line_number":46,"context_line":"            self.evaluator.run_evaluator()"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"            # if not self._wait_for_action(self.evaluator_queue.empty):"},{"line_number":49,"context_line":"            #     LOG.error(\u0027Evaluator Queue Not Empty\u0027)"},{"line_number":50,"context_line":"            #"},{"line_number":51,"context_line":"            # self._mark_old_deduced_alarms_as_deleted(timestamp, self.graph,"},{"line_number":52,"context_line":"            #                                          self.evaluator_queue)"},{"line_number":53,"context_line":"            self.status \u003d self.FINISHED"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"            LOG.info(\u0027Init Finished\u0027)"}],"source_content_type":"text/x-python","patch_set":4,"id":"df87a7cf_737e686c","line":52,"range":{"start_line":48,"start_character":0,"end_line":52,"end_character":76},"in_reply_to":"df87a7cf_776d0197","updated":"2017-12-05 07:56:14.000000000","message":"I added a comment in the code","commit_id":"4e71b45753447edaf51a0f328abf97ea2ca65582"}],"vitrage/tests/functional/entity_graph/consistency/test_consistency.py":[{"author":{"_account_id":21414,"name":"Yujun Zhang","email":"zhang.yujunz@zte.com.cn","username":"yujunz"},"change_message_id":"99da5d56e1a560222096614966480ebd29c92214","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        cls.event_queue \u003d queue.Queue()"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        def actions_callback(event_type, data):"},{"line_number":93,"context_line":"            cls.event_queue.put(data)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        scenario_repo \u003d ScenarioRepository(cls.conf)"}],"source_content_type":"text/x-python","patch_set":4,"id":"df87a7cf_17e27504","line":92,"range":{"start_line":92,"start_character":29,"end_line":92,"end_character":39},"updated":"2017-12-04 20:22:11.000000000","message":"This parameter is not used?","commit_id":"4e71b45753447edaf51a0f328abf97ea2ca65582"},{"author":{"_account_id":19184,"name":"Idan Hefetz","email":"idan.hefetz@nokia.com","username":"idan-hefetz"},"change_message_id":"3568124a7cf631f47ca1711b27ed11887d0b2fb6","unresolved":false,"context_lines":[{"line_number":89,"context_line":""},{"line_number":90,"context_line":"        cls.event_queue \u003d queue.Queue()"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":"        def actions_callback(event_type, data):"},{"line_number":93,"context_line":"            cls.event_queue.put(data)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        scenario_repo \u003d ScenarioRepository(cls.conf)"}],"source_content_type":"text/x-python","patch_set":4,"id":"df87a7cf_536fa491","line":92,"range":{"start_line":92,"start_character":29,"end_line":92,"end_character":39},"in_reply_to":"df87a7cf_17e27504","updated":"2017-12-05 07:56:14.000000000","message":"The event type is now always the same (and is unused), and so, ignored in the tests. It is now described in a comment","commit_id":"4e71b45753447edaf51a0f328abf97ea2ca65582"}],"vitrage/tests/unit/entity_graph/test_processor_service.py":[{"author":{"_account_id":19134,"name":"Eyal","email":"eyalb1@gmail.com","username":"eyalb"},"change_message_id":"c8a25c586d9fe80d87a694476730f393e4a74d49","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        the result should be the number of low priority calls."},{"line_number":41,"context_line":"        0*(2^n) + 1*n"},{"line_number":42,"context_line":"        \"\"\""},{"line_number":43,"context_line":"        tested_object \u003d TwoPriorityListener(None, self.do_work, None, None)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        def write_high():"},{"line_number":46,"context_line":"            time.sleep(1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_0b09f72a","line":43,"range":{"start_line":43,"start_character":8,"end_line":43,"end_character":21},"updated":"2017-12-04 14:37:06.000000000","message":"call it listener","commit_id":"caaaf634412933bdb127f2add650febc94f0d0cc"},{"author":{"_account_id":19134,"name":"Eyal","email":"eyalb1@gmail.com","username":"eyalb"},"change_message_id":"c8a25c586d9fe80d87a694476730f393e4a74d49","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        the result should be the number of low priority calls."},{"line_number":41,"context_line":"        0*(2^n) + 1*n"},{"line_number":42,"context_line":"        \"\"\""},{"line_number":43,"context_line":"        tested_object \u003d TwoPriorityListener(None, self.do_work, None, None)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        def write_high():"},{"line_number":46,"context_line":"            time.sleep(1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_cb122f59","line":43,"range":{"start_line":43,"start_character":44,"end_line":43,"end_character":75},"updated":"2017-12-04 14:37:06.000000000","message":"what is None ?\nmaybe make it a default parameter in the c\u0027tor","commit_id":"caaaf634412933bdb127f2add650febc94f0d0cc"},{"author":{"_account_id":19184,"name":"Idan Hefetz","email":"idan.hefetz@nokia.com","username":"idan-hefetz"},"change_message_id":"91615f7b7fa0482ebdae4070229efbbf5ddb0320","unresolved":false,"context_lines":[{"line_number":40,"context_line":"        the result should be the number of low priority calls."},{"line_number":41,"context_line":"        0*(2^n) + 1*n"},{"line_number":42,"context_line":"        \"\"\""},{"line_number":43,"context_line":"        tested_object \u003d TwoPriorityListener(None, self.do_work, None, None)"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"        def write_high():"},{"line_number":46,"context_line":"            time.sleep(1)"}],"source_content_type":"text/x-python","patch_set":2,"id":"df87a7cf_0ed105ce","line":43,"range":{"start_line":43,"start_character":44,"end_line":43,"end_character":75},"in_reply_to":"df87a7cf_cb122f59","updated":"2017-12-04 15:48:05.000000000","message":"It shouldn\u0027t be None, only for testing","commit_id":"caaaf634412933bdb127f2add650febc94f0d0cc"}]}