)]}'
{"heat/engine/resources/neutron/router.py":[{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"4420ce4cc13f35532deef18cd559ee0427e3ef35","unresolved":false,"context_lines":[{"line_number":351,"context_line":""},{"line_number":352,"context_line":"    def handle_delete(self):"},{"line_number":353,"context_line":"        if not self.resource_id:"},{"line_number":354,"context_line":"            return"},{"line_number":355,"context_line":"        client \u003d self.neutron()"},{"line_number":356,"context_line":"        router_id \u003d self.resource_id.split(\u0027:\u0027)[0]"},{"line_number":357,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_52267117","line":354,"updated":"2014-11-21 01:29:06.000000000","message":"If set the resource_id to router_id, the logic here is incorrect, the gateway_router won\u0027t be removed, right?","commit_id":"c7ba3770c3c4f9b0f8b2f6647ca43f9bf9db7adf"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"8823cb9b65fa2ad7ea4f9d1bce38548c78007b25","unresolved":false,"context_lines":[{"line_number":351,"context_line":""},{"line_number":352,"context_line":"    def handle_delete(self):"},{"line_number":353,"context_line":"        if not self.resource_id:"},{"line_number":354,"context_line":"            return"},{"line_number":355,"context_line":"        client \u003d self.neutron()"},{"line_number":356,"context_line":"        router_id \u003d self.resource_id.split(\u0027:\u0027)[0]"},{"line_number":357,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_7259ad5a","line":354,"in_reply_to":"5a890539_52267117","updated":"2014-11-21 01:49:16.000000000","message":"Sorry ignore this comment.","commit_id":"c7ba3770c3c4f9b0f8b2f6647ca43f9bf9db7adf"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"4420ce4cc13f35532deef18cd559ee0427e3ef35","unresolved":false,"context_lines":[{"line_number":353,"context_line":"        if not self.resource_id:"},{"line_number":354,"context_line":"            return"},{"line_number":355,"context_line":"        client \u003d self.neutron()"},{"line_number":356,"context_line":"        router_id \u003d self.resource_id.split(\u0027:\u0027)[0]"},{"line_number":357,"context_line":"        try:"},{"line_number":358,"context_line":"            client.remove_gateway_router(router_id)"},{"line_number":359,"context_line":"        except Exception as ex:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_52d111fa","line":356,"updated":"2014-11-21 01:29:06.000000000","message":"If the resource_id is router_id, why need split here?","commit_id":"c7ba3770c3c4f9b0f8b2f6647ca43f9bf9db7adf"},{"author":{"_account_id":8289,"name":"huangtianhua","email":"tiantian223@163.com","username":"huangtianhua"},"change_message_id":"a41a658ffd393f9d7eab0fac9d4ed146671553fc","unresolved":false,"context_lines":[{"line_number":353,"context_line":"        if not self.resource_id:"},{"line_number":354,"context_line":"            return"},{"line_number":355,"context_line":"        client \u003d self.neutron()"},{"line_number":356,"context_line":"        router_id \u003d self.resource_id.split(\u0027:\u0027)[0]"},{"line_number":357,"context_line":"        try:"},{"line_number":358,"context_line":"            client.remove_gateway_router(router_id)"},{"line_number":359,"context_line":"        except Exception as ex:"}],"source_content_type":"text/x-python","patch_set":3,"id":"5a890539_150a1321","line":356,"in_reply_to":"5a890539_52d111fa","updated":"2014-11-21 01:54:36.000000000","message":"Oh, I understand, it for compatibility with old data, would you please to add comment for this:)","commit_id":"c7ba3770c3c4f9b0f8b2f6647ca43f9bf9db7adf"}]}