john-bodley edited a comment on pull request #10427: URL: https://github.com/apache/incubator-superset/pull/10427#issuecomment-664016193
Note in terms of the failing tests I've reverted some of my changes in `tests/celery_tests.py` which removed some `session.close()` statements. Per the [Session.close](https://docs.sqlalchemy.org/en/13/orm/session_api.html#sqlalchemy.orm.session.Session.close) documentation, > This clears all items and ends any transaction in progress. which may be necessary as some of the Celery tasks first try to drop a table and may throw an exception if the table exists. Ending the transaction may be necessary prior to running the async query. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
