This is an automated email from the ASF dual-hosted git repository.
paksyd pushed a commit to branch branch-3
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-3 by this push:
new bfba4238fbc HBASE-29859 Initialization log for DataTieringManager is
incorrect (#7688)
bfba4238fbc is described below
commit bfba4238fbc0c5be03f3dc852b749faa64e9a9f5
Author: Liu Xiao <[email protected]>
AuthorDate: Wed Feb 4 21:32:35 2026 +0800
HBASE-29859 Initialization log for DataTieringManager is incorrect (#7688)
Signed-off-by: Dávid Paksy <[email protected]>
Reviewed-by: Vaibhav Joshi <[email protected]>
(cherry picked from commit 5c12752138f9ba4be938515e2e086e0693c4811c)
---
.../hadoop/hbase/regionserver/DataTieringManager.java | 16 ++++++++++------
1 file changed, 10 insertions(+), 6 deletions(-)
diff --git
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DataTieringManager.java
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DataTieringManager.java
index cb02c04e9e3..2c92d9238dc 100644
---
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DataTieringManager.java
+++
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/DataTieringManager.java
@@ -69,14 +69,18 @@ public class DataTieringManager {
*/
public static synchronized boolean instantiate(Configuration conf,
Map<String, HRegion> onlineRegions) {
- if (isDataTieringFeatureEnabled(conf) && instance == null) {
- instance = new DataTieringManager(onlineRegions);
- LOG.info("DataTieringManager instantiated successfully.");
- return true;
- } else {
+ if (!isDataTieringFeatureEnabled(conf)) {
+ LOG.debug("DataTiering feature is disabled (key: {}). Skipping
instantiation.",
+ GLOBAL_DATA_TIERING_ENABLED_KEY);
+ return false;
+ }
+ if (instance != null) {
LOG.warn("DataTieringManager is already instantiated.");
+ return false;
}
- return false;
+ instance = new DataTieringManager(onlineRegions);
+ LOG.info("DataTieringManager instantiated successfully.");
+ return true;
}
/**
