)]}'
{"neutron/services/segments/db.py":[{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"db6c808c668f033c73412abba9dd7e639ab42154","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        # Delete segment in DB"},{"line_number":178,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":179,"context_line":"            query \u003d self._model_query(context, segment_model.NetworkSegment)"},{"line_number":180,"context_line":"            seg \u003d query.filter(segment_model.NetworkSegment.id \u003d\u003d uuid).first()"},{"line_number":181,"context_line":"            if not seg:"},{"line_number":182,"context_line":"                raise exceptions.SegmentNotFound(segment_id\u003duuid)"},{"line_number":183,"context_line":"            context.session.delete(seg)"}],"source_content_type":"text/x-python","patch_set":9,"id":"dfeb2761_69086b2b","line":180,"updated":"2017-04-04 03:31:56.000000000","message":"I think they end up looking about the same either way.","commit_id":"e1b060188141de8f448f412db5e6e08ac395c532"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"6545069f8bc28f6f6347dfab4f366908333586ac","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        # Delete segment in DB"},{"line_number":178,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":179,"context_line":"            query \u003d self._model_query(context, segment_model.NetworkSegment)"},{"line_number":180,"context_line":"            seg \u003d query.filter(segment_model.NetworkSegment.id \u003d\u003d uuid).first()"},{"line_number":181,"context_line":"            if not seg:"},{"line_number":182,"context_line":"                raise exceptions.SegmentNotFound(segment_id\u003duuid)"},{"line_number":183,"context_line":"            context.session.delete(seg)"}],"source_content_type":"text/x-python","patch_set":9,"id":"7ffa3b31_df299f9b","line":180,"updated":"2017-04-17 23:01:21.000000000","message":"Yes","commit_id":"e1b060188141de8f448f412db5e6e08ac395c532"},{"author":{"_account_id":17120,"name":"Manjeet Singh Bhatia","email":"manjeet.s.bhatia@intel.com","username":"manjeets"},"change_message_id":"2850cb6a9d10243dace86760fd63ea034bd5422c","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        # Delete segment in DB"},{"line_number":178,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":179,"context_line":"            query \u003d self._model_query(context, segment_model.NetworkSegment)"},{"line_number":180,"context_line":"            seg \u003d query.filter(segment_model.NetworkSegment.id \u003d\u003d uuid).first()"},{"line_number":181,"context_line":"            if not seg:"},{"line_number":182,"context_line":"                raise exceptions.SegmentNotFound(segment_id\u003duuid)"},{"line_number":183,"context_line":"            context.session.delete(seg)"}],"source_content_type":"text/x-python","patch_set":9,"id":"dfeb2761_0bb7d7d7","line":180,"range":{"start_line":180,"start_character":17,"end_line":180,"end_character":71},"updated":"2017-03-31 21:04:07.000000000","message":"may be this can be simplified as well with filter_by ?","commit_id":"e1b060188141de8f448f412db5e6e08ac395c532"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"1e6334dc8adeecad8251cb931d2c7ba4e61b117b","unresolved":false,"context_lines":[{"line_number":177,"context_line":"        # Delete segment in DB"},{"line_number":178,"context_line":"        with context.session.begin(subtransactions\u003dTrue):"},{"line_number":179,"context_line":"            query \u003d self._model_query(context, segment_model.NetworkSegment)"},{"line_number":180,"context_line":"            seg \u003d query.filter(segment_model.NetworkSegment.id \u003d\u003d uuid).first()"},{"line_number":181,"context_line":"            if not seg:"},{"line_number":182,"context_line":"                raise exceptions.SegmentNotFound(segment_id\u003duuid)"},{"line_number":183,"context_line":"            context.session.delete(seg)"}],"source_content_type":"text/x-python","patch_set":9,"id":"7ffa3b31_36af90e4","line":180,"in_reply_to":"dfeb2761_69086b2b","updated":"2017-04-17 19:47:42.000000000","message":"With https://review.openstack.org/#/c/457339/1/neutron/services/segments/db.py in, wouldn\u0027t it make sense to use NetworkSegment.delete_objects(context, id\u003duuid)?","commit_id":"e1b060188141de8f448f412db5e6e08ac395c532"}]}