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

fix: DH-18395: Both fix PRs in one cherry-pick #6559

Merged
merged 2 commits into from
Jan 13, 2025

Conversation

rcaudy
Copy link
Member

@rcaudy rcaudy commented Jan 13, 2025

No description provided.

…le locations caused by serial unmanage (deephaven#6552)

* Refactor TableBackTableLocation infrastructure to support partitioning columns. Avoid unnecessary queuing.
* Refactor TableLocationSubscriptionBuffer to avoid the need to unmanage all buffered locations individually, as this is much more expensive than simply dropping all of them.
* Performance "unit test" for profiling the fix
… is called on multiple referents (deephaven#6557)

* Make all looped LivenessManager.unmanage calls use the bulk variant, and improve documentation for the single variant to prompt use of the bulk variant
* Document TableLocationSubscriptionBuffer.LocationUpdate a bit better
@rcaudy rcaudy added this to the 0.37.3 milestone Jan 13, 2025
@rcaudy rcaudy self-assigned this Jan 13, 2025
@devinrsmith devinrsmith merged commit c210e48 into deephaven:rc/v0.37.x Jan 13, 2025
28 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jan 13, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants