)]}'
{"neutron/plugins/bigswitch/servermanager.py":[{"author":{"_account_id":6659,"name":"Paul Michali","email":"pc@michali.net","username":"pcm"},"change_message_id":"69cd87ebe0f5024b7cdeb7f32f84dd434d774432","unresolved":false,"context_lines":[{"line_number":420,"context_line":"        good_first \u003d sorted(self.servers, key\u003dlambda x: x.failed)"},{"line_number":421,"context_line":"        first_response \u003d None"},{"line_number":422,"context_line":"        for active_server in good_first:"},{"line_number":423,"context_line":"            ret \u003d active_server.rest_call(action, resource, data, headers,"},{"line_number":424,"context_line":"                                          timeout,"},{"line_number":425,"context_line":"                                          reconnect\u003dself.always_reconnect,"},{"line_number":426,"context_line":"                                          hash_handler\u003dhash_handler)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ac7a964_298fc0cd","line":423,"updated":"2014-08-15 18:25:36.000000000","message":"Could reduce duplication, could do:\n    retries \u003d 0\n    while True:\n        ret \u003d active_server.rest_call(...)\n        if ret[0] !\u003d httplib.SERVICE_UNAVAILABLE or retries \u003e\u003d SERVICE_UNAVAILABLE_RETRY_COUNT:\n            break\n        time.sleep(SERVICE_UNAVAILABLE_RETRY_INTERVAL)\n        retries +\u003d 1","commit_id":"33cd9fb6bbbb336d6d8310f2f1a85f090bd62818"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"6ed7b9cd75bafd2cbc3ec2f09c9449e702062812","unresolved":false,"context_lines":[{"line_number":420,"context_line":"        good_first \u003d sorted(self.servers, key\u003dlambda x: x.failed)"},{"line_number":421,"context_line":"        first_response \u003d None"},{"line_number":422,"context_line":"        for active_server in good_first:"},{"line_number":423,"context_line":"            ret \u003d active_server.rest_call(action, resource, data, headers,"},{"line_number":424,"context_line":"                                          timeout,"},{"line_number":425,"context_line":"                                          reconnect\u003dself.always_reconnect,"},{"line_number":426,"context_line":"                                          hash_handler\u003dhash_handler)"}],"source_content_type":"text/x-python","patch_set":1,"id":"1abeadc6_40a191f6","line":423,"in_reply_to":"7ac7a964_0cabf21d","updated":"2014-08-23 22:04:25.000000000","message":"+1","commit_id":"33cd9fb6bbbb336d6d8310f2f1a85f090bd62818"},{"author":{"_account_id":490,"name":"Sumit Naiksatam","email":"sumitnaiksatam@gmail.com","username":"snaiksat"},"change_message_id":"652ce2f386313bfbcea77af1809d051ffd4bb64b","unresolved":false,"context_lines":[{"line_number":420,"context_line":"        good_first \u003d sorted(self.servers, key\u003dlambda x: x.failed)"},{"line_number":421,"context_line":"        first_response \u003d None"},{"line_number":422,"context_line":"        for active_server in good_first:"},{"line_number":423,"context_line":"            ret \u003d active_server.rest_call(action, resource, data, headers,"},{"line_number":424,"context_line":"                                          timeout,"},{"line_number":425,"context_line":"                                          reconnect\u003dself.always_reconnect,"},{"line_number":426,"context_line":"                                          hash_handler\u003dhash_handler)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ac7a964_0cabf21d","line":423,"in_reply_to":"7ac7a964_298fc0cd","updated":"2014-08-17 17:28:24.000000000","message":"yeah, this seems like a good suggestion to avoid code duplication...","commit_id":"33cd9fb6bbbb336d6d8310f2f1a85f090bd62818"},{"author":{"_account_id":11614,"name":"Koteswara Rao Kelam","email":"koteswara.kelam@hp.com","username":"koteswar"},"change_message_id":"21c15eee64e8e248fe05b1be1ad1e9aff2f16e17","unresolved":false,"context_lines":[{"line_number":74,"context_line":"# error messages"},{"line_number":75,"context_line":"NXNETWORK \u003d \u0027NXVNS\u0027"},{"line_number":76,"context_line":"SERVICE_UNAVAILABLE_RETRY_COUNT \u003d 3"},{"line_number":77,"context_line":"SERVICE_UNAVAILABLE_RETRY_INTERVAL \u003d 3"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"class RemoteRestError(exceptions.NeutronException):"}],"source_content_type":"text/x-python","patch_set":2,"id":"1abeadc6_de32470f","line":77,"updated":"2014-08-28 05:39:24.000000000","message":"can you prefix macros with \"HTTP_\" as these error codes are related to http?","commit_id":"aaec93079ff221ad2c5db7a8724ae2f06795488a"},{"author":{"_account_id":6659,"name":"Paul Michali","email":"pc@michali.net","username":"pcm"},"change_message_id":"26aafd5576942c486e44b004d11c4d06a2c23bcd","unresolved":false,"context_lines":[{"line_number":420,"context_line":"        good_first \u003d sorted(self.servers, key\u003dlambda x: x.failed)"},{"line_number":421,"context_line":"        first_response \u003d None"},{"line_number":422,"context_line":"        for active_server in good_first:"},{"line_number":423,"context_line":"            for x in xrange(0, HTTP_SERVICE_UNAVAILABLE_RETRY_COUNT + 1):"},{"line_number":424,"context_line":"                ret \u003d active_server.rest_call(action, resource, data, headers,"},{"line_number":425,"context_line":"                                              timeout,"},{"line_number":426,"context_line":"                                              reconnect\u003dself.always_reconnect,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_265c4725","line":423,"updated":"2014-09-02 13:56:56.000000000","message":"Don\u0027t need 0, which is the default (or you can go from 1 to retry count (instead of retry count + 1). Can use range(), as range is small.","commit_id":"7142c983fd3424d7e3dab279ab1f60393283a7ab"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"b090d2de2ad3bc4ee6645c28449d5e5cb195f0e9","unresolved":false,"context_lines":[{"line_number":420,"context_line":"        good_first \u003d sorted(self.servers, key\u003dlambda x: x.failed)"},{"line_number":421,"context_line":"        first_response \u003d None"},{"line_number":422,"context_line":"        for active_server in good_first:"},{"line_number":423,"context_line":"            for x in xrange(0, HTTP_SERVICE_UNAVAILABLE_RETRY_COUNT + 1):"},{"line_number":424,"context_line":"                ret \u003d active_server.rest_call(action, resource, data, headers,"},{"line_number":425,"context_line":"                                              timeout,"},{"line_number":426,"context_line":"                                              reconnect\u003dself.always_reconnect,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa98f980_09f82be4","line":423,"in_reply_to":"fa98f980_265c4725","updated":"2014-09-02 18:40:31.000000000","message":"Done","commit_id":"7142c983fd3424d7e3dab279ab1f60393283a7ab"}]}