michael-s-molina opened a new pull request, #24550: URL: https://github.com/apache/superset/pull/24550
### SUMMARY Fix a bug in https://github.com/apache/superset/pull/24488 that happens when running on SQL Lite. There was an event being registered inside another event and causing a `deque mutated during iteration error`. According to SQL Alchemy [docs](https://docs.sqlalchemy.org/en/20/core/event.html#sqlalchemy.event.listen): > The [listen()](https://docs.sqlalchemy.org/en/20/core/event.html#sqlalchemy.event.listen) function cannot be called at the same time that the target event is being run. This has implications for thread safety, and also means an event cannot be added from inside the listener function for itself. The list of events to be run are present inside of a mutable collection that can’t be changed during iteration. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF <img width="1785" alt="Screenshot 2023-06-29 at 09 35 53" src="https://github.com/apache/superset/assets/70410625/94a130fc-9ebd-4587-a34d-e54cd6b8202d";> ### TESTING INSTRUCTIONS Check that the above error does not happen when accessing a dashboard on SQL Lite. ### ADDITIONAL INFORMATION - [ ] Has associated issue: - [ ] Required feature flags: - [ ] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
