You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by GitBox <gi...@apache.org> on 2022/07/21 03:34:53 UTC
[GitHub] [incubator-kyuubi] cxzl25 commented on a diff in pull request #3023: Kyuubi Hive JDBC: Replace UGI-based Kerberos authentication w/ JAAS
cxzl25 commented on code in PR #3023:
URL: https://github.com/apache/incubator-kyuubi/pull/3023#discussion_r926222365
##########
kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiConnection.java:
##########
@@ -83,7 +82,6 @@ public class KyuubiConnection implements SQLConnection, KyuubiLoggable {
public static final Logger LOG = LoggerFactory.getLogger(KyuubiConnection.class.getName());
public static final String BEELINE_MODE_PROPERTY = "BEELINE_MODE";
public static final String HS2_PROXY_USER = "hive.server2.proxy.user";
- public static final String HS2_CLIENT_TOKEN = "hiveserver2ClientToken";
Review Comment:
n this PR we do not support the `delegationToken` authentication method, we will add it back in the subsequent PR #3096, right?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org