-
Notifications
You must be signed in to change notification settings - Fork 6
Closed
Labels
Description
This issue was auto-generated
PR: deephaven/deephaven-core#6347
Author: nbauernfeind
Original PR Body
Fixes #6039.
Fixes #6053.
The barrage viewport protocol has now changed:
- viewports are never sent included rows; all added rows are included
- added rows are position space from within the server-respected client requested viewport
- removed rows are position space from within the server-respected client requested viewport
- snapshots on viewports include removed rows to simplify client side processing
- barrage update metadata now explicitly includes current table size; viewports no longer track the source table's full rowset
viewport clients must infer shifts:
- removed rows are in pre-update space
- added rows are in post-update space
- the end-result viewport is retained rows + added rows (retained are the existing rows minus removed rows) where the adds are in the offsets specified by added rows
In this PR we'll bump the following dependencies:
- flatbuffer from 1.12.0 to 24.3.25
- arrow from 13.0.0 to 18.0.0
- protobuf 3.25.3 to 3.25.4
- barrage from 0.6.0 to 0.7.2 (noting that 0.7.x was released for this feature set)