eschutho commented on pull request #15403:
URL: https://github.com/apache/superset/pull/15403#issuecomment-876639038


   @koszti if you want to look at the error above, I may have some simple 
workaround suggestions for the alert. Since FF is going to alert anyway with no 
real meaningful message, we could go ahead and put a `confirm` message for all 
browsers to ask the person if they want us to close their db connection when 
they leave. That also somewhat takes care of the configuration issue because 
people will have a choice. @betodealmeida @yousoph any thoughts on that? 
   For the race condition on the client side, I think a good solution would be 
to check the query status again before displaying the results, but that may be 
something that we can do in a followup PR.


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

Reply via email to