Apache9 commented on a change in pull request #2497:
URL: https://github.com/apache/hbase/pull/2497#discussion_r501542365
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/EnableTableProcedure.java
##########
@@ -111,25 +104,16 @@ protected Flow executeFromState(final MasterProcedureEnv
env, final EnableTableS
// How many replicas do we currently have? Check regions returned
from
// in-memory state.
int currentMaxReplica = getMaxReplicaId(regionsOfTable);
-
- // Read the META table to know the number of replicas the table
currently has.
- // If there was a table modification on region replica count then
need to
- // adjust replica counts here.
- int replicasFound = TableName.isMetaTableName(this.tableName)?
Review comment:
It will be log in the line below but I do not see the necesity. It does
not effect the actual logic, if there are inconsistency, we should report it in
the report from CatalogJanitor and fix it with HBCK2? Just a simple log here
will just make users confusing.
----------------------------------------------------------------
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:
[email protected]
