)]}'
{"neutron/tests/unit/services/loadbalancer/drivers/radware/test_plugin_driver.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"d2023bdf15b6c1061b7c26fe2242364564ef0060","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                              None, None)"},{"line_number":233,"context_line":"                ]"},{"line_number":234,"context_line":"                self.rest_call_mock.assert_has_calls(calls, any_order\u003dTrue)"},{"line_number":235,"context_line":"                eventlet.greenthread.sleep(0)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"    def test_update_vip(self):"},{"line_number":238,"context_line":"        self.rest_call_mock.reset_mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2FRu8%3D","line":235,"updated":"2014-03-07 04:40:37.000000000","message":"Comment: it is better to have a comment why greenthread sleep is required here.\n\nQuestion: Is sleep(0) enough? It works well so far (according to Jenkins results), but if there is another greenthread switching in vip deletion I am afraid it becomes unsufficient.","commit_id":"3159d1bd0b8047d6b89d17f0ab63bfcc4a01943a"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e6a86126946ed6b2bdf9acbe064e7a53dce8f828","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                              None, None)"},{"line_number":233,"context_line":"                ]"},{"line_number":234,"context_line":"                self.rest_call_mock.assert_has_calls(calls, any_order\u003dTrue)"},{"line_number":235,"context_line":"                eventlet.greenthread.sleep(0)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"    def test_update_vip(self):"},{"line_number":238,"context_line":"        self.rest_call_mock.reset_mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2FP5Q%3D","line":235,"in_reply_to":"AAAAVn%2F%2FQKo%3D","updated":"2014-03-07 09:23:04.000000000","message":"Agree that using sleep(0) unless we see any problem.","commit_id":"3159d1bd0b8047d6b89d17f0ab63bfcc4a01943a"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"66e9864148d904bb0785281e927744dc4d6f1d85","unresolved":false,"context_lines":[{"line_number":232,"context_line":"                              None, None)"},{"line_number":233,"context_line":"                ]"},{"line_number":234,"context_line":"                self.rest_call_mock.assert_has_calls(calls, any_order\u003dTrue)"},{"line_number":235,"context_line":"                eventlet.greenthread.sleep(0)"},{"line_number":236,"context_line":""},{"line_number":237,"context_line":"    def test_update_vip(self):"},{"line_number":238,"context_line":"        self.rest_call_mock.reset_mock()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAVn%2F%2FQKo%3D","line":235,"in_reply_to":"AAAAVn%2F%2FRu8%3D","updated":"2014-03-07 08:49:53.000000000","message":"I think only context switch is needed.\nI don\u0027t like the idea of waiting some real time spans unless it\u0027s proven to be necessary.\n\nI\u0027ll add the comment","commit_id":"3159d1bd0b8047d6b89d17f0ab63bfcc4a01943a"}]}