From fb4d648d04b8c88a3e474cd15e201c54852e5dd4 Mon Sep 17 00:00:00 2001 From: Grace Guo Date: Tue, 22 Jun 2021 11:08:29 -0700 Subject: [PATCH] Revert "fix: SQL Lab show "Refetch Results" button while fetching new query results (#15109)" (#15301) This reverts commit 408d58f937a544190ca64d13942a6a7b04b95615. --- superset-frontend/src/SqlLab/reducers/sqlLab.js | 15 +-------------- 1 file changed, 1 insertion(+), 14 deletions(-) diff --git a/superset-frontend/src/SqlLab/reducers/sqlLab.js b/superset-frontend/src/SqlLab/reducers/sqlLab.js index ec8438140ac3..daa06a97c88d 100644 --- a/superset-frontend/src/SqlLab/reducers/sqlLab.js +++ b/superset-frontend/src/SqlLab/reducers/sqlLab.js @@ -517,20 +517,7 @@ export default function sqlLabReducer(state = {}, action) { if (changedQuery.changedOn > queriesLastUpdate) { queriesLastUpdate = changedQuery.changedOn; } - const prevState = state.queries[id].state; - const currentState = changedQuery.state; - newQueries[id] = { - ...state.queries[id], - ...changedQuery, - // race condition: - // because of async behavior, sql lab may still poll a couple of seconds - // when it started fetching or finished rendering results - state: - currentState === 'success' && - ['fetching', 'success'].includes(prevState) - ? prevState - : currentState, - }; + newQueries[id] = { ...state.queries[id], ...changedQuery }; change = true; } });