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

Modify OA doRehash() functions to properly debit rehashCredits #3211

Merged
merged 4 commits into from
Dec 16, 2022

Conversation

lbooker42
Copy link
Contributor

@lbooker42 lbooker42 commented Dec 15, 2022

This change prevents the rehash queue from being starved when there are frequent large updates.

Will close #3202

@lbooker42 lbooker42 requested review from cpwright and rcaudy December 15, 2022 20:30
@lbooker42 lbooker42 self-assigned this Dec 15, 2022
@lbooker42 lbooker42 added bug Something isn't working clean up labels Dec 15, 2022
@lbooker42 lbooker42 added this to the Dec 2022 milestone Dec 15, 2022
@lbooker42 lbooker42 changed the title Modify Natural Join doRehash() function to allow add'l rehash work each cycle Modify OA doRehash() functions to properly debit rehashCredits Dec 16, 2022
@lbooker42 lbooker42 merged commit 05c140e into deephaven:main Dec 16, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2022
@lbooker42 lbooker42 deleted the lab-nj-bugfix branch June 26, 2024 20:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working clean up NoDocumentationNeeded NoReleaseNotesNeeded No release notes are needed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Assertion failure in natural join update processing
2 participants