cxzl25 commented on code in PR #2691:
URL: https://github.com/apache/incubator-kyuubi/pull/2691#discussion_r875735597
##########
kyuubi-server/src/main/scala/org/apache/kyuubi/engine/KyuubiApplicationManager.scala:
##########
@@ -98,10 +100,9 @@ object KyuubiApplicationManager {
}
private def setupFlinkK8sTag(tag: String, conf: KyuubiConf): Unit = {
- // TODO: yarn.tags or flink.yarn.tags, the mess of flink settings confuses
me now.
Review Comment:
Check the Flink configuration documentation, there is no such configuration
as `flink.yarn.tags`.
Although Flink supports converting `flink.yarn.xx` to `yarn.xx`, there is no
`yarn.tags` configuration in yarn.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
