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.

Sign up for GitHub

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

Jump to bottom

JS API subscriptions should pass updateInterval to the server #186

Closed
niloc132 opened this issue Mar 23, 2021 · 0 comments · Fixed by #5890
Closed

JS API subscriptions should pass updateInterval to the server #186

niloc132 opened this issue Mar 23, 2021 · 0 comments · Fixed by #5890
Assignees
@niloc132
Labels
jsapi triage
Milestone
Backlog

Comments

@niloc132
Copy link
Member

Old wiring involved calling preemptiveUpdatesTable, so the JS API would have already set the updateInterval by the time the subscription was created.

Downside of not fixing this is that JS tables/charts are stuck at 1s per update.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@niloc132 niloc132

Labels
jsapi triage
Projects
None yet
Milestone
Backlog
Development

Successfully merging a pull request may close this issue.

refactor: Rewrite JS API to share server implementation of Flight/Barrage niloc132/deephaven-core
2 participants
@niloc132 @mofojed