Added disk spilling for merge#4218
Closed
thomasfrederikhoeck wants to merge 3 commits intodelta-io:mainfrom
Closed
Added disk spilling for merge#4218thomasfrederikhoeck wants to merge 3 commits intodelta-io:mainfrom
thomasfrederikhoeck wants to merge 3 commits intodelta-io:mainfrom
Conversation
Added disk spilling for merge similar to optimize functions
thomasfrederikhoeck
requested review from
hntd187,
ion-elgreco,
roeap and
rtyler
as code owners
github-actions
bot
added
binding/python
|
ACTION NEEDED delta-rs follows the Conventional Commits specification for release automation. The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification. |
added 2 commits
February 20, 2026 12:01
Added disk spilling for merge similar to optimize functions Signed-off-by: Thomas Frederik Hoeck <tfh@norden.com>
…oeck/delta-rs into feat_spill_merge
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added disk spilling for
mergesimilar to optimize functions to allow for merges which touches many files in the targetDescription
I have added functionality for spilling to disk similar to how it works in the optimize functions. If nothing is provided it works as before.
I have added test similar to those for the other spill functions.
I have tested my cases in #4217 which now successfully completes the merge without OOM.
I have used AI (Opus 4.6) for getting a overview of the project structure and for writing most of the code. I have review and verified the code myself.
Related Issue(s)
Closes #4217
Documentation