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

(Viewport) Support DoExchange equivalent of ConstructSnapshot over Flight #925

Closed
nbauernfeind opened this issue Jul 27, 2021 · 0 comments · Fixed by #1854
Closed

(Viewport) Support DoExchange equivalent of ConstructSnapshot over Flight #925

nbauernfeind opened this issue Jul 27, 2021 · 0 comments · Fixed by #1854
Assignees
@lbooker42
Labels
arrow barrage csv feature request New feature or request grpc
Milestone
Jan 2022

Comments

@nbauernfeind
Copy link
Member

nbauernfeind commented Jul 27, 2021 •
edited
Loading

It's a little silly, but Flight's DoGet does not allow a user to supply app_metadata. We can still support a DoGet w/column and viewport filters, but has to be tucked nicely inside of DoExchange.
@nbauernfeind nbauernfeind added this to the July 2021 milestone Jul 27, 2021
@nbauernfeind nbauernfeind self-assigned this Jul 27, 2021
@nbauernfeind nbauernfeind changed the title Support BarrageSubscriptionRequest app_metadata in Flight DoGet Support BarrageSubscriptionRequest app_metadata in Flight DoGet Jul 27, 2021
@nbauernfeind nbauernfeind modified the milestones: July 2021, Sept 2021 Aug 31, 2021
@nbauernfeind nbauernfeind changed the title Support BarrageSubscriptionRequest app_metadata in Flight DoGet Support DoExchange equivalent of ConstructSnapshot over Flight Aug 31, 2021
@nbauernfeind nbauernfeind modified the milestones: Sept 2021, Backlog Sep 8, 2021
@nbauernfeind nbauernfeind mentioned this issue Sep 8, 2021
Closed
@nbauernfeind nbauernfeind modified the milestones: Backlog, Nov 2021 Nov 16, 2021
@niloc132 niloc132 mentioned this issue Dec 10, 2021
Closed
@pete-petey pete-petey modified the milestones: Nov 2021, Jan 2022 Dec 29, 2021
@pete-petey pete-petey added the csv label Dec 29, 2021
@pete-petey pete-petey changed the title Support DoExchange equivalent of ConstructSnapshot over Flight (Viewport) Support DoExchange equivalent of ConstructSnapshot over Flight Jan 4, 2022
@rcaudy rcaudy mentioned this issue Jan 20, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lbooker42 lbooker42

Labels
arrow barrage csv feature request New feature or request grpc
Projects
None yet
Milestone
Jan 2022
Development

Successfully merging a pull request may close this issue.

Implemented viewport snapshot functionality over DoExchange lbooker42/deephaven-core
4 participants
@nbauernfeind @lbooker42 @rcaudy @pete-petey