Rename Hive access control 'none' to 'allow-all'

gerashegalov · Oct 22, 2015 · 30a4696 · 30a4696
1 parent b1b220d
commit 30a4696
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/presto-hive/src/main/java/com/facebook/presto/hive/HiveConnectorFactory.java b/presto-hive/src/main/java/com/facebook/presto/hive/HiveConnectorFactory.java
@@ -45,6 +45,7 @@
 import java.util.Map;
 
 import static com.facebook.presto.hive.ConditionalModule.installModuleIf;
+import static com.facebook.presto.hive.SecurityConfig.ALLOW_ALL_ACCESS_CONTROL;
 import static com.google.common.base.Preconditions.checkArgument;
 import static com.google.common.base.Strings.isNullOrEmpty;
 import static java.util.Objects.requireNonNull;
@@ -95,7 +96,7 @@ public Connector create(String connectorId, Map<String, String> config)
  new HiveClientModule(connectorId, metastore, typeManager, pageIndexerFactory),
  installModuleIf(
  SecurityConfig.class,
- security -> "none".equalsIgnoreCase(security.getSecuritySystem()),
+ security -> ALLOW_ALL_ACCESS_CONTROL.equalsIgnoreCase(security.getSecuritySystem()),
  new NoSecurityModule()),
  installModuleIf(
  SecurityConfig.class,

diff --git a/presto-hive/src/main/java/com/facebook/presto/hive/SecurityConfig.java b/presto-hive/src/main/java/com/facebook/presto/hive/SecurityConfig.java
@@ -17,7 +17,9 @@
 
 public class SecurityConfig
 {
- private String securitySystem = "none";
+ public static final String ALLOW_ALL_ACCESS_CONTROL = "allow-all";
+
+ private String securitySystem = ALLOW_ALL_ACCESS_CONTROL;
 
  public String getSecuritySystem()
  {