[GitHub] [hbase] saintstack commented on a change in pull request #465: HBASE-22808 HBCK Report showed the offline regions which belong to di…

[GitBox <gi...@apache.org>]

saintstack commented on a change in pull request #465: HBASE-22808 HBCK Report showed the offline regions which belong to di…
URL: https://github.com/apache/hbase/pull/465#discussion_r311845480
 
 

 ##########
 File path: hbase-server/src/main/java/org/apache/hadoop/hbase/master/CatalogJanitor.java
 ##########
 @@ -672,6 +672,11 @@ private void checkServer(byte [] metaTableRow, RegionLocations locations) {
         if (sn == null) {
           continue;
         }
+        // skip the offline regions which belong to disabled table.
+        if (services.getTableStateManager()
+            .isTableState(location.getRegion().getTable(), TableState.State.DISABLED)) {
 
 Review comment:
   Oh. I see. I didn't see it in my reproduction of the issue.
   
   Could be DISABLED or DISABLING. Thats what we check in other places IIRC.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services