Skip to content

HBASE-30248 Validate snapshot region timeout (#8468) - #8497

Merged
guluo2016 merged 1 commit into
apache:branch-2from
Sigma-Ma:HBASE-30248-branch-2
Jul 27, 2026
Merged

HBASE-30248 Validate snapshot region timeout (#8468)#8497
guluo2016 merged 1 commit into
apache:branch-2from
Sigma-Ma:HBASE-30248-branch-2

Conversation

@Sigma-Ma

Copy link
Copy Markdown
Contributor

(cherry picked from commit 4d417aa)

Backport of #8468 to branch-2, as requested in #8468 (comment).

Cherry-picked from commit 4d417aa. No branch-specific adaptations were required.

What changes were proposed in this pull request?

Validate hbase.snapshot.region.timeout and fall back to the documented default when the configured value is less than or equal to zero.

Why are the changes needed?

An invalid timeout can be passed to ThreadPoolExecutor, causing an IllegalArgumentException and aborting the RegionServer during startup.

How was this patch tested?

mvn -Dhadoop.profile=3.0 \
  -pl hbase-server -am \
  -DskipITs \
  -Dtest=TestRegionServerSnapshotManager \
  clean test

Signed-off-by: Peng Lu <lupeng@apache.org>
(cherry picked from commit 4d417aa)
@guluo2016 guluo2016 added the backport This PR is a back port of some issue or issues already committed to master label Jul 26, 2026

@guluo2016 guluo2016 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@guluo2016
guluo2016 merged commit dbd46bb into apache:branch-2 Jul 27, 2026
38 of 40 checks passed
guluo2016 pushed a commit that referenced this pull request Jul 27, 2026
Signed-off-by: Peng Lu <lupeng@apache.org>
guluo2016 pushed a commit that referenced this pull request Jul 27, 2026
Signed-off-by: Peng Lu <lupeng@apache.org>
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.

2 participants