)]}'
{"cinder/coordination.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"68f0dd1a36f6575523813f54b0c68e13d7c13c86","unresolved":false,"context_lines":[{"line_number":137,"context_line":"    def _synchronized(f, *a, **k):"},{"line_number":138,"context_line":"        call_args \u003d inspect.getcallargs(f, *a, **k)"},{"line_number":139,"context_line":"        call_args[\u0027f_name\u0027] \u003d f.__name__"},{"line_number":140,"context_line":"        lock \u003d coordinator.get_lock(lock_name.format(**call_args))"},{"line_number":141,"context_line":"        t1 \u003d timeutils.now()"},{"line_number":142,"context_line":"        t2 \u003d None"},{"line_number":143,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff0f0b1f_5f60dc01","line":140,"updated":"2017-05-19 09:24:34.000000000","message":"nit: It would be good to do the string type check like we were doing in L124","commit_id":"d7ef450e3cdb0b6b5eb133af18a779edd5c22ef1"}],"cinder/volume/manager.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"68f0dd1a36f6575523813f54b0c68e13d7c13c86","unresolved":false,"context_lines":[{"line_number":662,"context_line":"            if locked_action is None:"},{"line_number":663,"context_line":"                _run_flow()"},{"line_number":664,"context_line":"            else:"},{"line_number":665,"context_line":"                with coordination.COORDINATOR.get_lock(locked_action):"},{"line_number":666,"context_line":"                    _run_flow()"},{"line_number":667,"context_line":"        finally:"},{"line_number":668,"context_line":"            try:"}],"source_content_type":"text/x-python","patch_set":9,"id":"ff0f0b1f_420ebf32","line":665,"updated":"2017-05-19 09:24:34.000000000","message":"It looked better before, but it\u0027s negligible","commit_id":"d7ef450e3cdb0b6b5eb133af18a779edd5c22ef1"}]}