Skip to content

HBASE-30501 Preserve original HFiles when splitting for bulk load - #8866

Merged
Umeshkumar9414 merged 1 commit into
apache:branch-2from
ujjawal4046:HBASE-30501-branch-2
Nov 11, 2026
Merged

Umeshkumar9414 merged 1 commit into
apache:branch-2from
ujjawal4046:HBASE-30501-branch-2

Conversation

@ujjawal4046

Copy link
Copy Markdown
Contributor

Summary

Backports HBASE-30501 to branch-2.

Preserve original top-level HFiles after splitting so a failed bulk load can be retried by a fresh process. Only intermediate split inputs already under .tmp are deleted after being split again.

This is a manual adaptation because branch-2 still implements the logic in LoadIncrementalHFiles, while the master change was made in BulkLoadHFilesTool.

Upstream master PR: #8819

Test

mvn -pl hbase-server -PrunLargeTests -Dhadoop.profile=3.0 \
  -Dwarbucks.skip=true \
  -Dtest=TestLoadIncrementalHFilesSplitRecovery#testFreshRetryAfterSplitFailure \
  -Dcheckstyle.skip=true -Dspotless.check.skip=true test

Result: 1 test run, 0 failures, 0 errors, 0 skipped.

@liuxiaocs7 liuxiaocs7 added the backport This PR is a back port of some issue or issues already committed to master label Nov 10, 2026
@Umeshkumar9414
Umeshkumar9414 merged commit bdafa86 into apache:branch-2 Nov 11, 2026
22 of 25 checks passed
Umeshkumar9414 pushed a commit that referenced this pull request Nov 11, 2026
)

Signed-off by: Umesh Kumar Kumawat <ukumawat@apache.org>
Umeshkumar9414 pushed a commit that referenced this pull request Nov 11, 2026
)

Signed-off by: Umesh Kumar Kumawat <ukumawat@apache.org>
@Umeshkumar9414

Copy link
Copy Markdown
Contributor

I have cherry-picked it to branch-2.6 and branch-2.5 as well.
On a side note, If possible please refrain from using force push when addressing comments, it is not recommanded.

@Umeshkumar9414

Copy link
Copy Markdown
Contributor

Thanks @ujjawal4046 for your contribution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a back port of some issue or issues already committed to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants