)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":18816,"name":"Maurice Escher","display_name":"carthaca","email":"maurice.escher@sap.com","username":"mapocace"},"change_message_id":"7c0bd161adb11364818a725bb0b8a5b063d8e550","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"42295531_b31cdabd","updated":"2026-05-26 07:36:59.000000000","message":"See some minor remarks inline.\n\nTo me it looks like you may have used AI (no offense), if yes please add \u0027Assisted-By:\u0027 in the commit msg according to https://openinfra.org/legal/ai-policy","commit_id":"f274c1607d5f3b32dc83f42b35b8354b0723f3bc"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"0d76b93455977887d70594c6a4fd0a1a12449f33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4bb6dc5b_14baef80","in_reply_to":"42295531_b31cdabd","updated":"2026-05-26 08:18:42.000000000","message":"yes, AI help was used for comments, test cases etc. Added \u0027Assisted-By\u0027","commit_id":"f274c1607d5f3b32dc83f42b35b8354b0723f3bc"}],"manila/db/sqlalchemy/api.py":[{"author":{"_account_id":18816,"name":"Maurice Escher","display_name":"carthaca","email":"maurice.escher@sap.com","username":"mapocace"},"change_message_id":"7c0bd161adb11364818a725bb0b8a5b063d8e550","unresolved":true,"context_lines":[{"line_number":6730,"context_line":"                context.session.delete(access_map)"},{"line_number":6731,"context_line":"        except db_exc.DBError:"},{"line_number":6732,"context_line":"            LOG.warning("},{"line_number":6733,"context_line":"                \"Deleting soft-deleted resource %s failed, skipping.\","},{"line_number":6734,"context_line":"                access_map,"},{"line_number":6735,"context_line":"            )"},{"line_number":6736,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"e907ff36_45eae5eb","line":6733,"updated":"2026-05-26 07:36:59.000000000","message":"We can be more specific in the error message here, instead of \u0027resource\u0027 we can tell that it is about the \u0027access_map\u0027","commit_id":"f274c1607d5f3b32dc83f42b35b8354b0723f3bc"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"0d76b93455977887d70594c6a4fd0a1a12449f33","unresolved":false,"context_lines":[{"line_number":6730,"context_line":"                context.session.delete(access_map)"},{"line_number":6731,"context_line":"        except db_exc.DBError:"},{"line_number":6732,"context_line":"            LOG.warning("},{"line_number":6733,"context_line":"                \"Deleting soft-deleted resource %s failed, skipping.\","},{"line_number":6734,"context_line":"                access_map,"},{"line_number":6735,"context_line":"            )"},{"line_number":6736,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"a55e8d37_dfb1ef5e","line":6733,"in_reply_to":"e907ff36_45eae5eb","updated":"2026-05-26 08:18:42.000000000","message":"Done","commit_id":"f274c1607d5f3b32dc83f42b35b8354b0723f3bc"},{"author":{"_account_id":18816,"name":"Maurice Escher","display_name":"carthaca","email":"maurice.escher@sap.com","username":"mapocace"},"change_message_id":"7c0bd161adb11364818a725bb0b8a5b063d8e550","unresolved":true,"context_lines":[{"line_number":6771,"context_line":""},{"line_number":6772,"context_line":"    # Tables handled by dedicated purge helpers above; skip in the main loop"},{"line_number":6773,"context_line":"    # to avoid FK constraint errors from incorrect deletion ordering."},{"line_number":6774,"context_line":"    tables_with_dedicated_purge \u003d {\u0027share_access_map\u0027}"},{"line_number":6775,"context_line":""},{"line_number":6776,"context_line":"    for table in reversed(tables):"},{"line_number":6777,"context_line":"        if \u0027deleted\u0027 not in table.columns.keys():"}],"source_content_type":"text/x-python","patch_set":1,"id":"62db0375_466bd54a","line":6774,"updated":"2026-05-26 07:36:59.000000000","message":"I think it makes sense to also add \u0027share_network_security_service_association\u0027 here","commit_id":"f274c1607d5f3b32dc83f42b35b8354b0723f3bc"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"0d76b93455977887d70594c6a4fd0a1a12449f33","unresolved":false,"context_lines":[{"line_number":6771,"context_line":""},{"line_number":6772,"context_line":"    # Tables handled by dedicated purge helpers above; skip in the main loop"},{"line_number":6773,"context_line":"    # to avoid FK constraint errors from incorrect deletion ordering."},{"line_number":6774,"context_line":"    tables_with_dedicated_purge \u003d {\u0027share_access_map\u0027}"},{"line_number":6775,"context_line":""},{"line_number":6776,"context_line":"    for table in reversed(tables):"},{"line_number":6777,"context_line":"        if \u0027deleted\u0027 not in table.columns.keys():"}],"source_content_type":"text/x-python","patch_set":1,"id":"a5037a0b_798a1869","line":6774,"in_reply_to":"62db0375_466bd54a","updated":"2026-05-26 08:18:42.000000000","message":"Done","commit_id":"f274c1607d5f3b32dc83f42b35b8354b0723f3bc"}]}