-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat: query loading bars #22298
feat: query loading bars #22298
Conversation
…thog into hackathon_query_loading_bars
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh please 🚢 this
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
…thog into hackathon_query_loading_bars
Size Change: +97 B (+0.01%) Total Size: 1.05 MB ℹ️ View Unchanged
|
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
* printing results * sleep each row * in progress * returning loading status for single queries * cluster all * new schema * working row counts * change types and store in redis * loading * working redis * include and exclude * object creation * nan * Move some things around * WIP: loading bar * WIP: loading bar * summary * add better formatting * hooked up * add feature flag * fix numbers * fix nan * multiple queries * gate backend * remove sleep and print * Update UI snapshots for `chromium` (1) * Update UI snapshots for `chromium` (1) * python typing * dom props * Update UI snapshots for `chromium` (1) * validate * fix test * bunch of bugfixes * Update UI snapshots for `chromium` (1) * Update UI snapshots for `chromium` (1) * backend test * remove stories * fix type issues * Update UI snapshots for `chromium` (2) --------- Co-authored-by: Alexander Spicer <aspicer@surfer-172-30-3-221-hotspot.internet-for-guests.com> Co-authored-by: timgl <tim@posthog.com> Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Problem
Long running queries confuse users and they don't what's going on. The hedgehog running with no feedback gets frustrating.
Changes
Roll out a beta of a loading bar behind a feature flag
Screen.Recording.2024-05-15.at.7.47.47.PM.mov
How did you test this code?
Loading paths tested on local