You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by gr...@apache.org on 2019/05/16 22:22:31 UTC

[incubator-superset] branch master updated: [sql lab] Fix new query stuck at pending state (#7523)

This is an automated email from the ASF dual-hosted git repository.

graceguo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 7f858e4  [sql lab] Fix new query stuck at pending state (#7523)
7f858e4 is described below

commit 7f858e4566bd4944a904c0530709e0ba0a89eb5e
Author: Grace Guo <gr...@airbnb.com>
AuthorDate: Thu May 16 15:22:23 2019 -0700

    [sql lab] Fix new query stuck at pending state (#7523)
---
 superset/assets/src/SqlLab/components/QueryAutoRefresh.jsx | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/superset/assets/src/SqlLab/components/QueryAutoRefresh.jsx b/superset/assets/src/SqlLab/components/QueryAutoRefresh.jsx
index 8ad02ad..21d335b 100644
--- a/superset/assets/src/SqlLab/components/QueryAutoRefresh.jsx
+++ b/superset/assets/src/SqlLab/components/QueryAutoRefresh.jsx
@@ -38,7 +38,7 @@ class QueryAutoRefresh extends React.PureComponent {
   }
   shouldCheckForQueries() {
     // if there are started or running queries, this method should return true
-    const { queries, queriesLastUpdate } = this.props;
+    const { queries } = this.props;
     const now = new Date().getTime();
 
     // due to a race condition, queries can be marked as successful before the
@@ -50,7 +50,6 @@ class QueryAutoRefresh extends React.PureComponent {
     );
 
     return (
-      queriesLastUpdate > 0 &&
       Object.values(queries).some(
         q => isQueryRunning(q) &&
         now - q.startDttm < MAX_QUERY_AGE_TO_POLL,