yaooqinn commented on a change in pull request #989:
URL: https://github.com/apache/incubator-kyuubi/pull/989#discussion_r698129412
##########
File path:
kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
##########
@@ -735,4 +735,13 @@ object KyuubiConf {
.version("1.3.0")
.booleanConf
.createWithDefault(true)
+
+ val SESSION_NAME: OptionalConfigEntry[String] =
+ buildConf("session.name")
+ .doc("Session name is used to relate the history session which can help
optimize. " +
+ "Please set this config using different value for workflows which are
session level. " +
+ "This config is none means we can not find the history session.")
Review comment:
Session name is just a session name. History is just a typical use case,
the doc is overkilling to me
--
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]
