pan3793 commented on code in PR #6383: URL: https://github.com/apache/kyuubi/pull/6383#discussion_r1806536773
########## 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) Review Comment: ~can we let `kyuubi.engine.flink.doAs.enabled` default value fallback to `kyuubi.engine.doAs.enabled`, and make flink engine exclusively respect `kyuubi.engine.flink.doAs.enabled`?~ ignore this. -- 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