pan3793 commented on code in PR #7043: URL: https://github.com/apache/kyuubi/pull/7043#discussion_r2059564216
########## kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala: ########## @@ -1951,6 +1951,16 @@ object KyuubiConf { .stringConf .createWithDefault("1") + val BATCH_INFO_INTERNAL_REDIRECT = buildConf("kyuubi.batch.info.internal.redirect") + .serverOnly + .doc("When set to true, the batch info is retrieved by forwarding the request to the " + + "corresponding Kyuubi instance using the internal REST client. If false, the batch info is " + Review Comment: I remember there may have been some state inconsistent risk for batch v1 implementation previously, is that still 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