)]}'
{"neutron/plugins/ml2/plugin.py":[{"author":{"_account_id":1689,"name":"Robert Kukura","email":"rk@theep.net","username":"rkukura"},"change_message_id":"8c6d6dca929e86810a321a69fa513936bf070be8","unresolved":false,"context_lines":[{"line_number":606,"context_line":"            # the fact that an error occurred."},{"line_number":607,"context_line":"            LOG.error(_(\"mechanism_manager.delete_subnet_postcommit failed\"))"},{"line_number":608,"context_line":""},{"line_number":609,"context_line":"    @lockutils.do_not_cooperatively_yield()"},{"line_number":610,"context_line":"    def create_port(self, context, port):"},{"line_number":611,"context_line":"        attrs \u003d port[\u0027port\u0027]"},{"line_number":612,"context_line":"        attrs[\u0027status\u0027] \u003d const.PORT_STATUS_DOWN"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FCzI%3D","line":609,"updated":"2014-03-20 17:48:03.000000000","message":"The create_port() function calls create_port_postcommit() on mechanism drivers, which can block for significant periods of time communicating with controllers or devices. If this decorator prevents other greenthreads from running while waiting for those responses, I think this will be very counterproductive. Maybe it could instead be applied to just the transaction (line 615).","commit_id":"3af0fbb1989258b0dc07ec19cde6bd3f2aa9d99d"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"5b2d6d3006c89a1931617035581418987cf6d38f","unresolved":false,"context_lines":[{"line_number":606,"context_line":"            # the fact that an error occurred."},{"line_number":607,"context_line":"            LOG.error(_(\"mechanism_manager.delete_subnet_postcommit failed\"))"},{"line_number":608,"context_line":""},{"line_number":609,"context_line":"    @lockutils.do_not_cooperatively_yield()"},{"line_number":610,"context_line":"    def create_port(self, context, port):"},{"line_number":611,"context_line":"        attrs \u003d port[\u0027port\u0027]"},{"line_number":612,"context_line":"        attrs[\u0027status\u0027] \u003d const.PORT_STATUS_DOWN"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FCfI%3D","line":609,"in_reply_to":"AAAAWH%2F%2FCzI%3D","updated":"2014-03-20 18:25:52.000000000","message":"Definitely agree. This is just a work in progress. I put the decorator there to see how many things it breaks in the various testing environments.","commit_id":"3af0fbb1989258b0dc07ec19cde6bd3f2aa9d99d"}]}
