hive-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From pxi...@apache.org
Subject [3/7] hive git commit: HIVE-8594 : Wrong condition in SettableConfigUpdater#setHiveConfWhiteList() (Ted Yu via Szehon)
Date Tue, 11 Aug 2015 21:12:01 GMT
HIVE-8594 : Wrong condition in SettableConfigUpdater#setHiveConfWhiteList() (Ted Yu via Szehon)

git-svn-id: https://svn.apache.org/repos/asf/hive/trunk@1640844 13f79535-47bb-0310-9956-ffa450edef68


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/dbdea20d
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/dbdea20d
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/dbdea20d

Branch: refs/heads/branch-1.0
Commit: dbdea20d22a9c64d450299406f74f13b8f0b7ac3
Parents: 6be7f51
Author: Szehon Ho <szehon@apache.org>
Authored: Fri Nov 21 00:53:34 2014 +0000
Committer: Pengcheng Xiong <pxiong@apache.org>
Committed: Tue Aug 11 13:55:23 2015 -0700

----------------------------------------------------------------------
 .../ql/security/authorization/plugin/SettableConfigUpdater.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/dbdea20d/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java
b/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java
index 89f155c..f12cd51 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java
@@ -44,7 +44,7 @@ public class SettableConfigUpdater {
     String whiteListParamsStr = hiveConf
         .getVar(ConfVars.HIVE_AUTHORIZATION_SQL_STD_AUTH_CONFIG_WHITELIST);
 
-    if(whiteListParamsStr == null && whiteListParamsStr.trim().isEmpty()) {
+    if(whiteListParamsStr == null || whiteListParamsStr.trim().isEmpty()) {
       throw new HiveAuthzPluginException("Configuration parameter "
           + ConfVars.HIVE_AUTHORIZATION_SQL_STD_AUTH_CONFIG_WHITELIST.varname
           + " is not iniatialized.");


Mime
View raw message