rusackas commented on PR #34111: URL: https://github.com/apache/superset/pull/34111#issuecomment-4722211195
Thanks for digging into this one, @LucasWolke - the diagnosis was spot on, and sorry it sat so long. I opened #41127 to carry it across the line: same root cause, but passing `is_split=not run_multiple_statements_as_one` so it stays correct for `MUTATE_AFTER_SPLIT=False` folks too, plus the same fix in the new Celery executor and tests for both paths. Went with a fresh PR rather than pushing to your branch mostly so I could fold in @giacomochiarella's use case and the extra coverage without trampling your history... you're credited as co-author there. That one supersedes this, so it'll close once #41127 lands. Thanks again for sticking with it! :) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
