-
Notifications
You must be signed in to change notification settings - Fork 174
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
Add option to disable live queries #1052
Comments
Oh, I get it now! That's a big change from UX perspective. But I like how you made it backwards compatible. Probably, in new mode we would grey out visualisation/pinboard after changes and enable "update now" (or just play icon) button. That also affects url - should we change url when someone changes filters/splits but doesn't update visualisation? That sounds like a lot of |
The fact is that only few users notice update now button in refresh menu and click F5 or CTRL-R which cost more load on servers.
My colleagues say that pinboard and filters should keep refresh on every filter change because it is crucial in qube query construction. We could gray out results instead.
I think that url should change and reflect filter/split/measure/display state - if you refresh page it should load results. |
I'm getting more and more convinced. Few notes.
|
I won't argue, but it's related so it could be in this place.
Personally I would not store live query mode in local storage - I'd like to have different behaviour on different browser tabs. I'd like to force users to disable live queries on very big, heavy datacubes - so I'd prefer to have this setting in configuration on datacube level.
That is great idea, but I still recommend pulling "update now" button from auto update menu. |
Oh, I didn't think about it. Makes sense. Probably every admin of turnilo already rolled out some config generator script so repetition there wouldn't be a problem but granularity is definitely a win.
Yeah, button for "Evaluating" changed filters will be in the middle. "Update now" button require some rethinking. We'd like that users will embrace browser - if you need to refresh stale tab, just use browser refresh button. That will refresh everything, not only data but data cube max time which can impact time filter calendar view and who knows what else. Out of curiosity, do you use auto refresh on any of your cubes? |
Time filter should update itself on every form display (#527)
It's very rare. I can't remember if I used it last year. |
upvote, can someone please implement this feature? |
@on99 - thank you for the PR, I would like to discuss proposed change with the team in Allegro. I'll let you know. |
Sometimes druid response is slow and it would be better to disable live queries that refresh result view while building the final filters/split/measure/view set to prevent overloading druid with unneccessary queries.
I propose to:
I'd appreciate feedback and discussion.
The text was updated successfully, but these errors were encountered: