pan3793 commented on code in PR #6383: URL: https://github.com/apache/kyuubi/pull/6383#discussion_r1806590337
########## kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala: ########## @@ -2999,6 +2999,24 @@ object KyuubiConf { .version("1.8.1") .fallbackConf(ENGINE_INITIALIZE_SQL) + val ENGINE_FLINK_DOAS_ENABLED: ConfigEntry[Boolean] = + buildConf("kyuubi.engine.flink.doAs.enabled") + .doc("When enabled, the session user is used as the proxy user to launch the Flink engine," + + " otherwise, the server user. Note, due to the limitation of Apache Flink," + + " it can only be enabled on Kerberized environment.") + .version("1.10.0") + .booleanConf + .createWithDefault(false) + + val ENGINE_FLINK_DOAS_GENERATE_TOKEN_FILE: ConfigEntry[Boolean] = + buildConf("kyuubi.engine.flink.doAs.generateTokenFile") + .doc("Whether to generate a hadoop token file for flink submit process." + + s" We need to enable it when we set `$ENGINE_FLINK_DOAS_ENABLED=true`" + Review Comment: this should be an internal flag -we can remove it anytime I think the issue could also be addressed by YARN-10333 (correct me if I'm wrong), so the docs might be "When `${ENGINE_FLINK_DOAS_ENABLED.key}=true` and neither FLINK-35525 (Flink 1.20.0) nor YARN-10333 (Hadoop 3.4.0) is available, enable this configuration to generate a temporary HADOOP_TOKEN_FILE that will be picked up by the Flink engine bootstrap process." -- 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-unsubscr...@kyuubi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@kyuubi.apache.org For additional commands, e-mail: notifications-h...@kyuubi.apache.org