smacker commented on issue #7611: [sqllab] Cancel database query on stop
URL: 
https://github.com/apache/incubator-superset/pull/7611#issuecomment-525676234
 
 
   @mistercrunch no. #8097 stops executing list of queries if one of them was 
stopped.
   
   This PR stops (kills) underlying connection to DB (no matter 1 or more 
queries are there).
   In async mode in SQLLab queries are executed in celery and stop button 
doesn't really stop them, the task is running until it gets some results from 
DB. (which may take days)
   
   If you are interested in the fix, I can re-work it using `extra_json` 
instead of a new field in DB. It will make easier to add support for other 
backends. (for example for Spark just id isn't enough but special object should 
be serialized: https://github.com/src-d/sourced-ui/pull/267)

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to