You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While reviewing #6347 it became clear how much simpler it would be to review this particular piece of code if the interface was split into at least two view-like implementations, possibly three.
Full Subscriptions should have their own entry point.
Viewport Subscriptions should have their own entry point.
Possibly Growing Viewports should have their own entry point.
We also need a design document, deephaven.io level documentation, or perhaps just very good javadoc in this file, describing the three strategies and how one can expect the RecordBatch to be structured and populated.
The text was updated successfully, but these errors were encountered:
While reviewing #6347 it became clear how much simpler it would be to review this particular piece of code if the interface was split into at least two view-like implementations, possibly three.
We also need a design document, deephaven.io level documentation, or perhaps just very good javadoc in this file, describing the three strategies and how one can expect the RecordBatch to be structured and populated.
The text was updated successfully, but these errors were encountered: