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

"Reverse Viewport" support in the Deephaven server and Java client #1946

Closed
rcaudy opened this issue Feb 7, 2022 · 0 comments · Fixed by #1968
Closed

"Reverse Viewport" support in the Deephaven server and Java client #1946

rcaudy opened this issue Feb 7, 2022 · 0 comments · Fixed by #1968
Assignees
Labels
Milestone

Comments

@rcaudy
Copy link
Member

rcaudy commented Feb 7, 2022

As a user, I would like to be able to efficiently subscribe to a range of rows at the end of an updating table. Currently, this requires a 3-way dance:

  1. Server notifies client that the table's size has changed
  2. Client asks the server to move the viewport to the "new" end
  3. Server computes and sends a new effective viewport and snapshot to the client

This introduced extra message traffic and latency for viewport changes.

Alternatively, with a "reverse viewport" subscription, the server will know that the client wants to stay subscribed to rows starting from the end of the table as position 0, and counting up towards the first row of the table. This will allow the server to simply skip straight to step 3, advising the client of its new effective viewport and snapshot, with no explicit request from the client.

We have already added the necessary request message support in Barrage 0.4.0. This ticket is to cover the implementation on the server side and in the Java reference client.

@rcaudy rcaudy added feature request New feature or request triage labels Feb 7, 2022
@rcaudy rcaudy added this to the Feb 2022 milestone Feb 7, 2022
mofojed pushed a commit that referenced this issue Apr 24, 2024
Release notes https://github.com/deephaven/web-client-ui/releases/tag/v0.73.0

# [0.73.0](deephaven/web-client-ui@v0.72.0...v0.73.0) (2024-04-19)


### Bug Fixes

* allow plotly plots to shrink inside ui.flex/grid layouts ([#1946](deephaven/web-client-ui#1946)) ([88fbe86](deephaven/web-client-ui@88fbe86))
* Fix issues when auto-size columns/rows is false, and when row headers are not 0 ([#1927](deephaven/web-client-ui#1927)) ([01c2a06](deephaven/web-client-ui@01c2a06))


### Features

* improve table loading ([#1898](deephaven/web-client-ui#1898)) ([9b14ee0](deephaven/web-client-ui@9b14ee0)), closes [#1865](deephaven/web-client-ui#1865)
* ListView components ([#1919](deephaven/web-client-ui#1919)) ([b63ab18](deephaven/web-client-ui@b63ab18))
* log export blacklist ([#1881](deephaven/web-client-ui#1881)) ([d3fb28a](deephaven/web-client-ui@d3fb28a)), closes [#1245](deephaven/web-client-ui#1245)
* New chart error panel ([#1850](deephaven/web-client-ui#1850)) ([309ff79](deephaven/web-client-ui@309ff79)), closes [#1520](deephaven/web-client-ui#1520)
* reopen closed tabs ([#1912](deephaven/web-client-ui#1912)) ([c2e8714](deephaven/web-client-ui@c2e8714)), closes [#1785](deephaven/web-client-ui#1785)


### BREAKING CHANGES

* `LIST_VIEW_ROW_HEIGHT` number constant replaced with
dictionary `LIST_VIEW_ROW_HEIGHTS`

Co-authored-by: deephaven-internal <66694643+deephaven-internal@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants