pan3793 commented on code in PR #6383: URL: https://github.com/apache/kyuubi/pull/6383#discussion_r1806537338
########## 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: ```suggestion s" We need to enable it when we set `${ENGINE_FLINK_DOAS_ENABLED.key}=true`" + ``` -- 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