virajjasani commented on a change in pull request #982: HBASE-23636 Disable table may hang when regionserver stop or abort. URL: https://github.com/apache/hbase/pull/982#discussion_r363018205
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/procedure/ServerCrashProcedure.java ########## @@ -482,13 +482,18 @@ private void assignRegions(MasterProcedureEnv env, List<RegionInfo> regions) thr regionNode.getProcedure().serverCrashed(env, regionNode, getServerName()); continue; } - if (env.getMasterServices().getTableStateManager().isTableState(regionNode.getTable(), - TableState.State.DISABLING, TableState.State.DISABLED)) { + if (env.getMasterServices().getTableStateManager() + .isTableState(regionNode.getTable(), TableState.State.DISABLING)) { env.getAssignmentManager().regionClosedAbnormally(regionNode); - LOG.info("{} found table disabled for region {}, set it state to ABNORMALLY_CLOSED.", + LOG.info("{} found table disabling for region {}, set it state to ABNORMALLY_CLOSED.", this, regionNode); continue; } + if (env.getMasterServices().getTableStateManager() + .isTableState(regionNode.getTable(), TableState.State.DISABLED)) { + LOG.warn("{} found table disabled for region {}.", this, regionNode); Review comment: `this` would print `server`, `splitWal` and `meta` correct? or it will print `pid`, `ppid`, `state` etc additionally? ---------------------------------------------------------------- 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