hbase-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anoopsamj...@apache.org
Subject svn commit: r1551816 - /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
Date Wed, 18 Dec 2013 03:10:10 GMT
Author: anoopsamjohn
Date: Wed Dec 18 03:10:08 2013
New Revision: 1551816

URL: http://svn.apache.org/r1551816
Log:
HBASE-10187 AccessController#preOpen - Include 'labels' table also into special tables list.

Modified:
    hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java

Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java?rev=1551816&r1=1551815&r2=1551816&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
(original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/security/access/AccessController.java
Wed Dec 18 03:10:08 2013
@@ -1014,7 +1014,7 @@ public class AccessController extends Ba
       LOG.error("NULL region from RegionCoprocessorEnvironment in preOpen()");
     } else {
       HRegionInfo regionInfo = region.getRegionInfo();
-      if (isSpecialTable(regionInfo)) {
+      if (regionInfo.getTable().isSystemTable()) {
         isSystemOrSuperUser(regionEnv.getConfiguration());
       } else {
         requirePermission("preOpen", Action.ADMIN);
@@ -1627,13 +1627,6 @@ public class AccessController extends Ba
     }
   }
 
-  private boolean isSpecialTable(HRegionInfo regionInfo) {
-    TableName tableName = regionInfo.getTable();
-    return tableName.equals(AccessControlLists.ACL_TABLE_NAME)
-        || tableName.equals(TableName.NAMESPACE_TABLE_NAME)
-        || tableName.equals(TableName.META_TABLE_NAME);
-  }
-
   @Override
   public void preStopRegionServer(
       ObserverContext<RegionServerCoprocessorEnvironment> env)



Mime
View raw message