)]}'
{"nova/rpc/impl_fake.py":[{"author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"change_message_id":"eae1608247b13a8044125647ec2d8b09bd48a6c0","unresolved":false,"context_lines":[{"line_number":91,"context_line":"                eventlet.greenthread.sleep(1)"},{"line_number":92,"context_line":"                cur_time \u003d time.time()"},{"line_number":93,"context_line":"                if (cur_time - start_time) \u003e timeout:"},{"line_number":94,"context_line":"                    raise rpc_common.Timeout()"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        return done.wait()"},{"line_number":97,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEn%2F%2F%2F3U%3D","line":94,"updated":"2012-02-02 16:08:24.000000000","message":"I guess this is only for the test suite, but wouldn\u0027t this leak the thread?","commit_id":"55abf3360d7594aabeef7e7f2b9fdc55b2da57d6"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"61d34f22d2e7eb839815c8d23a79d1fd80e39f7c","unresolved":false,"context_lines":[{"line_number":91,"context_line":"                eventlet.greenthread.sleep(1)"},{"line_number":92,"context_line":"                cur_time \u003d time.time()"},{"line_number":93,"context_line":"                if (cur_time - start_time) \u003e timeout:"},{"line_number":94,"context_line":"                    raise rpc_common.Timeout()"},{"line_number":95,"context_line":""},{"line_number":96,"context_line":"        return done.wait()"},{"line_number":97,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEn%2F%2F%2Fz0%3D","line":94,"in_reply_to":"AAAAEn%2F%2F%2F3U%3D","updated":"2012-02-02 17:12:49.000000000","message":"Yeah, I didn\u0027t worry about it much since it\u0027s for the tests, but I\u0027ll add killing the thread here.","commit_id":"55abf3360d7594aabeef7e7f2b9fdc55b2da57d6"}],"nova/rpc/impl_qpid.py":[{"author":{"_account_id":100,"name":"Johannes Erdfelt","email":"johannes@erdfelt.com","username":"johannes.erdfelt"},"change_message_id":"eae1608247b13a8044125647ec2d8b09bd48a6c0","unresolved":false,"context_lines":[{"line_number":25,"context_line":"import qpid.messaging"},{"line_number":26,"context_line":"import qpid.messaging.exceptions"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"from nova import exception"},{"line_number":29,"context_line":"from nova.common import cfg"},{"line_number":30,"context_line":"from nova import flags"},{"line_number":31,"context_line":"from nova.rpc import amqp as rpc_amqp"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEn%2F%2F%2F3c%3D","line":28,"updated":"2012-02-02 16:08:24.000000000","message":"This doesn\u0027t appear to be used?","commit_id":"55abf3360d7594aabeef7e7f2b9fdc55b2da57d6"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"61d34f22d2e7eb839815c8d23a79d1fd80e39f7c","unresolved":false,"context_lines":[{"line_number":25,"context_line":"import qpid.messaging"},{"line_number":26,"context_line":"import qpid.messaging.exceptions"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"from nova import exception"},{"line_number":29,"context_line":"from nova.common import cfg"},{"line_number":30,"context_line":"from nova import flags"},{"line_number":31,"context_line":"from nova.rpc import amqp as rpc_amqp"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAEn%2F%2F%2F1Q%3D","line":28,"in_reply_to":"AAAAEn%2F%2F%2F3c%3D","updated":"2012-02-02 17:12:49.000000000","message":"You\u0027re right.  I had nova.rpc.common.Timeout() there at first, but moved it later.  I\u0027ll remove it.","commit_id":"55abf3360d7594aabeef7e7f2b9fdc55b2da57d6"}]}