[GitHub] [hbase] gentlewangyu commented on a change in pull request #1478: HBASE-24137:The max merge count of metafixer may be remind in hbase-site.xml

[GitBox]

gentlewangyu commented on a change in pull request #1478: HBASE-24137:The max 
merge count of metafixer may be remind in hbase-site.xml
URL: https://github.com/apache/hbase/pull/1478#discussion_r406648144
 
 

 ##########
 File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/master/MetaFixer.java
 ##########
 @@ -63,7 +61,7 @@
   MetaFixer(MasterServices masterServices) {
     this.masterServices = masterServices;
     this.maxMergeCount = this.masterServices.getConfiguration().
-        getInt(MAX_MERGE_COUNT_KEY, MAX_MERGE_COUNT_DEFAULT);
+        getInt("hbase.master.metafixer.max.merge.count", 10);
 
 Review comment:
   I think should be add hbase.master.metafixer.max.merge.count in 
hbase-default.xml that so operators can change it easier. I have added fuller 
description in hbase-default.xml.
   
   Thanks

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services