)]}'
{"quantum/plugins/nicira/nicira_qos_db.py":[{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"4ebfbb81b2567194b8e7237af4b48257a56a21bb","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        query \u003d self._model_query(context, NetworkQueueMapping)"},{"line_number":135,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":136,"context_line":"            binding \u003d query.filter_by(network_id\u003dnetwork_id).first()"},{"line_number":137,"context_line":"            if not binding:"},{"line_number":138,"context_line":"                # return since this can happen if we are updating a port that"},{"line_number":139,"context_line":"                # did not already have a queue on it. There is no need to check"},{"line_number":140,"context_line":"                # if there is one before deleting if we return here."}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2FeFY%3D","line":137,"updated":"2013-05-08 18:08:25.000000000","message":"Not sure why the negative logic needed. Consider:\n\nif binding:\n    context.session.delete(binding)","commit_id":"5d089e39890f4fb9b3ea21f972e6d8a11ce49e6d"},{"author":{"_account_id":6849,"name":"Roman Podoliaka","email":"roman.podoliaka@gmail.com","username":"rpodolyaka"},"change_message_id":"62cdcf05654aacb6536781a6c6b68307338c97e8","unresolved":false,"context_lines":[{"line_number":134,"context_line":"        query \u003d self._model_query(context, NetworkQueueMapping)"},{"line_number":135,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":136,"context_line":"            binding \u003d query.filter_by(network_id\u003dnetwork_id).first()"},{"line_number":137,"context_line":"            if not binding:"},{"line_number":138,"context_line":"                # return since this can happen if we are updating a port that"},{"line_number":139,"context_line":"                # did not already have a queue on it. There is no need to check"},{"line_number":140,"context_line":"                # if there is one before deleting if we return here."}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAOn%2F%2Fd4g%3D","line":137,"in_reply_to":"AAAAOn%2F%2FeFY%3D","updated":"2013-05-08 19:41:25.000000000","message":"Done","commit_id":"5d089e39890f4fb9b3ea21f972e6d8a11ce49e6d"}]}