You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hive.apache.org by sz...@apache.org on 2014/11/21 01:53:34 UTC

svn commit: r1640844 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java

Author: szehon
Date: Fri Nov 21 00:53:34 2014
New Revision: 1640844

URL: http://svn.apache.org/r1640844
Log:
HIVE-8594 : Wrong condition in SettableConfigUpdater#setHiveConfWhiteList() (Ted Yu via Szehon)

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java?rev=1640844&r1=1640843&r2=1640844&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/security/authorization/plugin/SettableConfigUpdater.java Fri Nov 21 00:53:34 2014
@@ -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.");