Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Revert "fix: SQL Lab show "Refetch Results" button while fetching new query results" #15301

Merged
merged 1 commit into from Jun 22, 2021

Conversation

graceguo-supercat
Copy link

Reverts #15109

This PR fixed the "Refetch Results" button as expected, but it seems it introduced another issue in airbnb. Some of our users saw "offline" message when they start new query. I am not sure the root cause yet, but have to revert to PR and investigate more.

@graceguo-supercat graceguo-supercat changed the title Revert "fix: SQL Lab show "Refetch Results" button while fetching new query results" fix: Revert "fix: SQL Lab show "Refetch Results" button while fetching new query results" Jun 22, 2021
@codecov
Copy link

codecov bot commented Jun 22, 2021

Codecov Report

Merging #15301 (05d1725) into master (4d48f04) will increase coverage by 0.00%.
The diff coverage is 0.00%.

❗ Current head 05d1725 differs from pull request most recent head 9177c6c. Consider uploading reports for the commit 9177c6c to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master   #15301   +/-   ##
=======================================
  Coverage   77.24%   77.25%           
=======================================
  Files         973      973           
  Lines       50438    50432    -6     
  Branches     6175     6171    -4     
=======================================
  Hits        38959    38959           
+ Misses      11274    11268    -6     
  Partials      205      205           
Flag Coverage Δ
javascript 71.81% <0.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
superset-frontend/src/SqlLab/reducers/sqlLab.js 34.95% <0.00%> (+0.83%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4d48f04...9177c6c. Read the comment docs.

@graceguo-supercat graceguo-supercat merged commit 5b2eb8d into master Jun 22, 2021
cccs-RyanS pushed a commit to CybercentreCanada/superset that referenced this pull request Dec 17, 2021
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 29, 2021
cccs-rc pushed a commit to CybercentreCanada/superset that referenced this pull request Mar 6, 2024
@mistercrunch mistercrunch deleted the revert-15109-gg-FixSqlLabResults branch March 26, 2024 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants