Kota-SH commented on code in PR #7857:
URL: https://github.com/apache/hbase/pull/7857#discussion_r2906965299
##########
hbase-common/src/main/java/org/apache/hadoop/hbase/TableName.java:
##########
@@ -92,7 +92,7 @@ public static TableName getDefaultNameOfMetaForReplica() {
public static TableName initializeHbaseMetaTableName(Configuration conf) {
String suffix_val = conf.get(HConstants.HBASE_META_TABLE_SUFFIX,
HConstants.HBASE_META_TABLE_SUFFIX_DEFAULT_VALUE);
- LOG.info("Meta table suffix value: {}", suffix_val);
+ LOG.info("Read Replica Cluster suffix value: {}", suffix_val);
Review Comment:
Perhaps we could move this log into the else block (line 98)? That way it
only fires when the suffix exists. If not, adding a prefix like "[Read-replica
feature]" to the log line would also work well.
Since we’re initializing the meta table name here, I think having "meta
table name suffix" in the log would be a bit more informative, wdyt?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
