From 8701290d0a5c9119b5830223accde57017f0b446 Mon Sep 17 00:00:00 2001 From: Junseung Hwang Date: Fri, 5 Apr 2019 22:37:13 +0900 Subject: [PATCH] modify RangerHiveAuthorizerBase to extend AbstractHiveAuthorizer, instead of implementing HiveAuthorizer directly --- .../hive/authorizer/RangerHiveAuthorizerBase.java | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizerBase.java b/hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizerBase.java index c313870cc1..d097d6a005 100644 --- a/hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizerBase.java +++ b/hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizerBase.java @@ -26,9 +26,9 @@ import org.apache.hadoop.hive.conf.HiveConf; import org.apache.hadoop.hive.conf.HiveConf.ConfVars; import org.apache.hadoop.hive.ql.security.HiveAuthenticationProvider; +import org.apache.hadoop.hive.ql.security.authorization.plugin.AbstractHiveAuthorizer; import org.apache.hadoop.hive.ql.security.authorization.plugin.DisallowTransformHook; import org.apache.hadoop.hive.ql.security.authorization.plugin.HiveAccessControlException; -import org.apache.hadoop.hive.ql.security.authorization.plugin.HiveAuthorizer; import org.apache.hadoop.hive.ql.security.authorization.plugin.HiveAuthzPluginException; import org.apache.hadoop.hive.ql.security.authorization.plugin.HiveAuthzSessionContext; import org.apache.hadoop.hive.ql.security.authorization.plugin.HiveAuthzSessionContext.CLIENT_TYPE; @@ -41,7 +41,7 @@ import org.apache.hadoop.security.UserGroupInformation; import org.apache.ranger.authorization.utils.StringUtil; -public abstract class RangerHiveAuthorizerBase implements HiveAuthorizer { +public abstract class RangerHiveAuthorizerBase extends AbstractHiveAuthorizer { private static final Log LOG = LogFactory.getLog(RangerHiveAuthorizerBase.class); @@ -217,12 +217,8 @@ public void setCurrentRole(String roleName) throwNotImplementedException("setCurrentRole"); } - public Object getHiveAuthorizationTranslator() throws HiveAuthzPluginException { - return null; - } - private void throwNotImplementedException(String method) throws HiveAuthzPluginException { - throw new HiveAuthzPluginException(method + "() not implemented in Ranger HiveAuthorizer"); + throw new HiveAuthzPluginException(method + "() not implemented in Ranger AbstractHiveAuthorizer"); } }