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

Posted to dev@drill.apache.org by sohami <gi...@git.apache.org> on 2018/02/06 08:16:32 UTC

[GitHub] drill pull request #1111: DRILL-5978: Upgrade Hive libraries to 2.1.1 versio...

Github user sohami commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1111#discussion_r166202560
  
    --- Diff: contrib/storage-hive/core/src/test/java/org/apache/drill/exec/impersonation/hive/TestSqlStdBasedAuthorization.java ---
    @@ -100,7 +103,9 @@ private static void setSqlStdBasedAuthorizationInHiveConf() {
         hiveConfig.put(METASTORE_EXECUTE_SET_UGI.varname, hiveConf.get(METASTORE_EXECUTE_SET_UGI.varname));
         hiveConfig.put(HIVE_AUTHORIZATION_ENABLED.varname, hiveConf.get(HIVE_AUTHORIZATION_ENABLED.varname));
         hiveConfig.put(HIVE_AUTHENTICATOR_MANAGER.varname, SessionStateUserAuthenticator.class.getName());
    -    hiveConfig.put(HIVE_AUTHORIZATION_MANAGER.varname, SQLStdHiveAuthorizerFactory.class.getName());
    --- End diff --
    
    why is this removed ? The test seem to be for Authorization.


---