HBASE-29837 Backport HBASE-27355 Separate meta read requests from master and client (#7261) by Umeshkumar9414 · Pull Request #7652 · apache/hbase

Umeshkumar9414 · 2026-01-20T13:30:04Z

Because, MetaTableAccessor moved to hbase-server in master/branch-3 in HBASE-24609 and HBASE-25758, cherry-pick of HBASE-27355 was not clean.

This is an important improvement and I think needed in 2.x as well.

…e#7261) Co-authored-by: huiruan <huiruan@tencent.com> Signed-off-by: Duo Zhang <zhangduo@apache.org> Reviewed-by: Aman Poonia <aman.poonia.29@gmail.com>

Umeshkumar9414 · 2026-01-20T13:39:41Z

 public static final int HIGH_QOS = 200;
 public static final int SYSTEMTABLE_QOS = HIGH_QOS;
+ // QOS for internal meta read requests
+ public static final int INTERNAL_READ_QOS = 250;


I also thought of another approch where we can define INTERNAL_READ_QOS in AnnotationReadingPriorityFunction (hbase-server module) and then let the master pass the priority while requesting to MetaTableAccessor, but that way we would end up doing many changes.

Umeshkumar9414 · 2026-01-20T13:44:26Z

@frostruan , @Apache9 , @mnpoonia please review.

Umeshkumar9414 · 2026-01-20T19:02:58Z

Same tests, TestSecureIPC, were passing in the previous build. So I think we can ignore Test failure.

Apache-HBase · 2026-01-20T19:22:59Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 57s		Docker mode activated.
			_ Prechecks _
+1 💚	dupname	0m 0s		No case conflicting files found.
+0 🆗	codespell	0m 0s		codespell was not available.
+0 🆗	detsecrets	0m 0s		detect-secrets was not available.
+1 💚	@author	0m 0s		The patch does not contain any @author tags.
+1 💚	hbaseanti	0m 0s		Patch does not have any anti-patterns.
			_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 15s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 55s		branch-2 passed
+1 💚	compile	5m 59s		branch-2 passed
+1 💚	checkstyle	1m 25s		branch-2 passed
+1 💚	spotbugs	3m 36s		branch-2 passed
+1 💚	spotless	1m 24s		branch has no errors when running spotless:check.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 18s		Maven dependency ordering for patch
+1 💚	mvninstall	4m 18s		the patch passed
+1 💚	compile	5m 33s		the patch passed
+1 💚	javac	5m 33s		the patch passed
+1 💚	blanks	0m 0s		The patch has no blanks issues.
+1 💚	checkstyle	1m 24s		the patch passed
+1 💚	spotbugs	4m 48s		the patch passed
+1 💚	hadoopcheck	23m 15s		Patch does not cause any errors with Hadoop 2.10.2 or 3.3.6 3.4.1.
+1 💚	spotless	1m 4s		patch has no errors when running spotless:check.
			_ Other Tests _
+1 💚	asflicense	0m 40s		The patch does not generate ASF License warnings.
		63m 9s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/artifact/yetus-general-check/output/Dockerfile
GITHUB PR	#7652
Optional Tests	dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
uname	Linux d5b194c33f8e 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `5ffb430`
Default Java	Eclipse Adoptium-11.0.23+9
Max. process+thread count	77 (vs. ulimit of 30000)
modules	C: hbase-common hbase-client hbase-server U: .
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/console
versions	git=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2026-01-20T22:50:26Z

💔 -1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	1m 34s		Docker mode activated.
-0 ⚠️	yetus	0m 5s		Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
			_ Prechecks _
			_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 11s		Maven dependency ordering for branch
+1 💚	mvninstall	2m 13s		branch-2 passed
+1 💚	compile	1m 2s		branch-2 passed
+1 💚	javadoc	0m 43s		branch-2 passed
+1 💚	shadedjars	3m 49s		branch has no errors when building our shaded downstream artifacts.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 11s		Maven dependency ordering for patch
+1 💚	mvninstall	1m 50s		the patch passed
+1 💚	compile	1m 0s		the patch passed
+1 💚	javac	1m 0s		the patch passed
+1 💚	javadoc	0m 42s		the patch passed
+1 💚	shadedjars	3m 47s		patch has no errors when building our shaded downstream artifacts.
			_ Other Tests _
+1 💚	unit	1m 45s		hbase-common in the patch passed.
+1 💚	unit	7m 30s		hbase-client in the patch passed.
-1 ❌	unit	239m 13s	/patch-unit-hbase-server.txt	hbase-server in the patch failed.
		270m 12s

Subsystem	Report/Notes
Docker	ClientAPI=1.48 ServerAPI=1.48 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR	#7652
Optional Tests	javac javadoc unit compile shadedjars
uname	Linux d6083d1b9b21 6.8.0-1024-aws #26~22.04.1-Ubuntu SMP Wed Feb 19 06:54:57 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `5ffb430`
Default Java	Temurin-1.8.0_412-b08
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/testReport/
Max. process+thread count	3338 (vs. ulimit of 30000)
modules	C: hbase-common hbase-client hbase-server U: .
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/console
versions	git=2.34.1 maven=3.9.8
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2026-01-20T22:59:38Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	0m 57s		Docker mode activated.
-0 ⚠️	yetus	0m 6s		Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
			_ Prechecks _
			_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 19s		Maven dependency ordering for branch
+1 💚	mvninstall	3m 48s		branch-2 passed
+1 💚	compile	1m 53s		branch-2 passed
+1 💚	javadoc	1m 14s		branch-2 passed
+1 💚	shadedjars	6m 24s		branch has no errors when building our shaded downstream artifacts.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 21s		Maven dependency ordering for patch
+1 💚	mvninstall	3m 11s		the patch passed
+1 💚	compile	1m 27s		the patch passed
+1 💚	javac	1m 27s		the patch passed
+1 💚	javadoc	0m 59s		the patch passed
+1 💚	shadedjars	6m 1s		patch has no errors when building our shaded downstream artifacts.
			_ Other Tests _
+1 💚	unit	2m 32s		hbase-common in the patch passed.
+1 💚	unit	8m 6s		hbase-client in the patch passed.
+1 💚	unit	236m 50s		hbase-server in the patch passed.
		279m 52s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR	#7652
Optional Tests	javac javadoc unit compile shadedjars
uname	Linux 18084680cf01 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `5ffb430`
Default Java	Eclipse Adoptium-11.0.23+9
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/testReport/
Max. process+thread count	3278 (vs. ulimit of 30000)
modules	C: hbase-common hbase-client hbase-server U: .
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/console
versions	git=2.34.1 maven=3.9.8
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

Apache-HBase · 2026-01-20T23:17:39Z

🎊 +1 overall

Vote	Subsystem	Runtime	Logfile	Comment
+0 🆗	reexec	1m 17s		Docker mode activated.
-0 ⚠️	yetus	0m 6s		Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
			_ Prechecks _
			_ branch-2 Compile Tests _
+0 🆗	mvndep	0m 14s		Maven dependency ordering for branch
+1 💚	mvninstall	4m 44s		branch-2 passed
+1 💚	compile	2m 17s		branch-2 passed
+1 💚	javadoc	1m 26s		branch-2 passed
+1 💚	shadedjars	7m 14s		branch has no errors when building our shaded downstream artifacts.
			_ Patch Compile Tests _
+0 🆗	mvndep	0m 19s		Maven dependency ordering for patch
+1 💚	mvninstall	4m 21s		the patch passed
+1 💚	compile	2m 15s		the patch passed
+1 💚	javac	2m 15s		the patch passed
+1 💚	javadoc	1m 27s		the patch passed
+1 💚	shadedjars	7m 8s		patch has no errors when building our shaded downstream artifacts.
			_ Other Tests _
+1 💚	unit	3m 14s		hbase-common in the patch passed.
+1 💚	unit	8m 3s		hbase-client in the patch passed.
+1 💚	unit	246m 17s		hbase-server in the patch passed.
		297m 48s

Subsystem	Report/Notes
Docker	ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR	#7652
Optional Tests	javac javadoc unit compile shadedjars
uname	Linux 29325373184e 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build tool	maven
Personality	dev-support/hbase-personality.sh
git revision	branch-2 / `5ffb430`
Default Java	Eclipse Adoptium-17.0.11+9
Test Results	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/testReport/
Max. process+thread count	3104 (vs. ulimit of 30000)
modules	C: hbase-common hbase-client hbase-server U: .
Console output	https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7652/2/console
versions	git=2.34.1 maven=3.9.8
Powered by	Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

Umeshkumar9414 · 2026-01-21T06:23:43Z

@apurtell , @Apache9 can you please review and merge.

apurtell

+1, will merge soon unless objection

…ter and client (#7261) (#7652) * HBASE-27355 Separate meta read requests from master and client (#7261) Co-authored-by: huiruan <huiruan@tencent.com> Signed-off-by: Duo Zhang <zhangduo@apache.org> Reviewed-by: Aman Poonia <aman.poonia.29@gmail.com> * HBASE-29837 spotless apply Signed-off-by: Duo Zhang <zhangduo@apache.org> Co-authored-by: Ruanhui <32773751+frostruan@users.noreply.github.com> Co-authored-by: ukumawat <ukumawat@salesforce.com> Signed-off-by: Andrew Purtell <apurtell@apache.org>

HBASE-27355 Separate meta read requests from master and client (apach…
12c05d5
…e#7261) Co-authored-by: huiruan <huiruan@tencent.com> Signed-off-by: Duo Zhang <zhangduo@apache.org> Reviewed-by: Aman Poonia <aman.poonia.29@gmail.com>

Umeshkumar9414 mentioned this pull request Jan 20, 2026
HBASE-29141 Calculate default maxQueueLength call queues correctly #7620
Closed

Umeshkumar9414 commented Jan 20, 2026
View reviewed changes

This comment has been minimized.
Sign in to view

HBASE-29837 spotless apply
5ffb430

This comment has been minimized.
Sign in to view

d-c-manning approved these changes Jan 21, 2026
View reviewed changes

frostruan approved these changes Jan 21, 2026
View reviewed changes

mnpoonia approved these changes Jan 21, 2026
View reviewed changes

apurtell approved these changes Jan 21, 2026
View reviewed changes

Umeshkumar9414 mentioned this pull request Jan 22, 2026
HBASE-29141 Calculate default maxQueueLength call queues correctly #7490
Merged

apurtell merged commit 5ad076d into apache:branch-2Jan 26, 2026
1 check failed

Navigation Menu

Uh oh!

HBASE-29837 Backport HBASE-27355 Separate meta read requests from master and client (#7261) - #7652

HBASE-29837 Backport HBASE-27355 Separate meta read requests from master and client (#7261)#7652
apurtell merged 2 commits into
apache:branch-2from
Umeshkumar9414:HBASE-29837

Umeshkumar9414 commented Jan 20, 2026

Uh oh!

Umeshkumar9414Jan 20, 2026

Uh oh!

Umeshkumar9414 commented Jan 20, 2026

Uh oh!

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Umeshkumar9414 commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Umeshkumar9414 commented Jan 21, 2026

Uh oh!

apurtell left a comment •
edited
Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Uh oh!

Conversation

Umeshkumar9414 commented Jan 20, 2026

Uh oh!

Umeshkumar9414Jan 20, 2026

Choose a reason for hiding this comment

Uh oh!

Umeshkumar9414 commented Jan 20, 2026

Uh oh!

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Umeshkumar9414 commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Apache-HBase commented Jan 20, 2026

Uh oh!

Umeshkumar9414 commented Jan 21, 2026

Uh oh!

apurtell left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

apurtell left a comment •
edited
Loading