Vitor-Avila commented on PR #29117: URL: https://github.com/apache/superset/pull/29117#issuecomment-2163374316
@john-bodley 100% agreed! This is currently happening because the statement to delete entries from the `tagged_object` table does not include a filter for the `object_type` column, so SQLAlchemy was expecting to delete x rows, but the operation would actually delete more than that. I worked on this as an attempt to have a fast fix in place (the issue is pretty simple to reproduce, make sure you have a dashboard and a chart with the same ID, assign tags/owners to both and try to delete one). Since it seems this solution didn't fully solve the issue just yet, I just submitted an alternative approach that I believe is better: https://github.com/apache/superset/pull/29229 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
