)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"3bcf7dd1a9d54fc8e4b8b95bcbd1a4ca6ced4582","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a63259de_dfc5a8d8","updated":"2022-07-14 12:37:42.000000000","message":"LGTM","commit_id":"99457e3079a4965b901ce7a049808d2717fcfd23"}],"cinder/db/sqlalchemy/api.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"99787c99d85f8deee3ded7772f1257bf7cf14423","unresolved":true,"context_lines":[{"line_number":37,"context_line":"from oslo_utils import timeutils"},{"line_number":38,"context_line":"from oslo_utils import uuidutils"},{"line_number":39,"context_line":"osprofiler_sqlalchemy \u003d importutils.try_import(\u0027osprofiler.sqlalchemy\u0027)"},{"line_number":40,"context_line":"import sqlalchemy as sa"},{"line_number":41,"context_line":"from sqlalchemy import MetaData"},{"line_number":42,"context_line":"from sqlalchemy import or_, and_"},{"line_number":43,"context_line":"from sqlalchemy.orm import joinedload, undefer_group, load_only"}],"source_content_type":"text/x-python","patch_set":3,"id":"f2431adf_f85b6e04","line":40,"updated":"2022-07-12 09:57:05.000000000","message":"this unnecessary change creates 3 additional unnecessary changes to the file.","commit_id":"99457e3079a4965b901ce7a049808d2717fcfd23"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"0ef682e5fae1dfc6c0fef6f29f3adab9ca240096","unresolved":false,"context_lines":[{"line_number":37,"context_line":"from oslo_utils import timeutils"},{"line_number":38,"context_line":"from oslo_utils import uuidutils"},{"line_number":39,"context_line":"osprofiler_sqlalchemy \u003d importutils.try_import(\u0027osprofiler.sqlalchemy\u0027)"},{"line_number":40,"context_line":"import sqlalchemy as sa"},{"line_number":41,"context_line":"from sqlalchemy import MetaData"},{"line_number":42,"context_line":"from sqlalchemy import or_, and_"},{"line_number":43,"context_line":"from sqlalchemy.orm import joinedload, undefer_group, load_only"}],"source_content_type":"text/x-python","patch_set":3,"id":"50c77ec0_f035e228","line":40,"in_reply_to":"f2431adf_f85b6e04","updated":"2022-07-14 12:37:59.000000000","message":"You\u0027re right. I think I was expecting to copy-paste more stuff from nova (in later patches) and wanted to avoid rewriting it. Not sure that actually happened. In any case, I should probably get rid of the below imports at some point since we\u0027re only supposed to import modules.","commit_id":"99457e3079a4965b901ce7a049808d2717fcfd23"}]}