You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2019/05/21 03:43:54 UTC

[GitHub] [incubator-superset] betodealmeida commented on issue #7559: Fix for polling queries

betodealmeida commented on issue #7559: Fix for polling queries
URL: https://github.com/apache/incubator-superset/pull/7559#issuecomment-494228154
 
 
   > Is there no way to make the state = success atomic along with the resultsKey affectation on the backend? The problem here is that success doesn't really mean "ok it's all done here!"
   
   @mistercrunch in theory it should be. I fixed an eager commit that created a race condition, but even after deploying the fix @vylc reported it still happening. I was hoping it was a a bad node that didn't pick up the change, but I checked all hosts and they were indeed running the fix, which is why I made this workaround.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org