dpgaspar commented on pull request #13350:
URL: https://github.com/apache/superset/pull/13350#issuecomment-786726126


   > @dpgaspar the `app.db` instance doesn't appear to use the null pool with 
the metadata database. I know we're using that elsewhere to get around this 
forking issue, which I'm planning on reviewing after this fix is merged.
   
   workers when accessing the metadata db should use: 
https://github.com/apache/superset/blob/master/superset/utils/celery.py#L33
   
   Let's talk about this?


----------------------------------------------------------------
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]

Reply via email to