)]}'
{"dragonflow/controller/ryu_base_app.py":[{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"b9b689d21b5fe25d0f0082ad4806768d91d98215","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                                           constants.CONTROLLER_REINITIALIZE,"},{"line_number":105,"context_line":"                                           None)"},{"line_number":106,"context_line":"        self.first_connect \u003d False"},{"line_number":107,"context_line":"        if self.vswitch_api:"},{"line_number":108,"context_line":"            self.vswitch_api.initialize(self.nb_api)"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    def _send_port_desc_stats_request(self, datapath):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_437d9e6c","line":107,"range":{"start_line":107,"start_character":16,"end_line":107,"end_character":27},"updated":"2017-07-11 11:36:47.000000000","message":"When it can be None?","commit_id":"a53c7a7d2338ceffd80cd40f7ad7bfcce12979ca"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"ca81110a9f9b4c03fd551ed47812554b5b106eb8","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                                           constants.CONTROLLER_REINITIALIZE,"},{"line_number":105,"context_line":"                                           None)"},{"line_number":106,"context_line":"        self.first_connect \u003d False"},{"line_number":107,"context_line":"        if self.vswitch_api:"},{"line_number":108,"context_line":"            self.vswitch_api.initialize(self.nb_api)"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    def _send_port_desc_stats_request(self, datapath):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f1a1f67_51047817","line":107,"range":{"start_line":107,"start_character":16,"end_line":107,"end_character":27},"in_reply_to":"1f1a1f67_e8d85780","updated":"2017-07-13 05:13:57.000000000","message":"Done. https://review.openstack.org/483194","commit_id":"a53c7a7d2338ceffd80cd40f7ad7bfcce12979ca"},{"author":{"_account_id":23766,"name":"Dima Kuznetsov","email":"dima.kuznetsov@toganetworks.com","username":"dimak"},"change_message_id":"2707f33cc920c54bacc829d4d40588160e1891cb","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                                           constants.CONTROLLER_REINITIALIZE,"},{"line_number":105,"context_line":"                                           None)"},{"line_number":106,"context_line":"        self.first_connect \u003d False"},{"line_number":107,"context_line":"        if self.vswitch_api:"},{"line_number":108,"context_line":"            self.vswitch_api.initialize(self.nb_api)"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    def _send_port_desc_stats_request(self, datapath):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1f1a1f67_e8d85780","line":107,"range":{"start_line":107,"start_character":16,"end_line":107,"end_character":27},"in_reply_to":"3f1d235d_2b6cb834","updated":"2017-07-12 11:04:40.000000000","message":"Reasonable enough. Maybe we can make it non-optional in another patch. If the None option is for testing, we can just use a mock.","commit_id":"a53c7a7d2338ceffd80cd40f7ad7bfcce12979ca"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"5318197512b97c9331d51ae66588c7dbc842a90b","unresolved":false,"context_lines":[{"line_number":104,"context_line":"                                           constants.CONTROLLER_REINITIALIZE,"},{"line_number":105,"context_line":"                                           None)"},{"line_number":106,"context_line":"        self.first_connect \u003d False"},{"line_number":107,"context_line":"        if self.vswitch_api:"},{"line_number":108,"context_line":"            self.vswitch_api.initialize(self.nb_api)"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    def _send_port_desc_stats_request(self, datapath):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f1d235d_2b6cb834","line":107,"range":{"start_line":107,"start_character":16,"end_line":107,"end_character":27},"in_reply_to":"3f1d235d_437d9e6c","updated":"2017-07-11 13:23:03.000000000","message":"Specifically, this happens in the tests. However, __init__ api allows it to be non-specified, defaulting to None. So it\u0027s either changing the API as well, or testing like this.","commit_id":"a53c7a7d2338ceffd80cd40f7ad7bfcce12979ca"}]}