SteNicholas commented on a change in pull request #1967:
URL: https://github.com/apache/incubator-kyuubi/pull/1967#discussion_r813505997
##########
File path:
kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala
##########
@@ -1148,6 +1148,19 @@ object KyuubiConf {
.checkValues(OperationModes.values.map(_.toString))
.createWithDefault(OperationModes.NONE.toString)
+ val OPERATION_PLAN_ONLY_EXCLUDES: ConfigEntry[Seq[String]] =
+ buildConf("operation.plan.only.excludes")
+ .doc("Comma-separated list of query plan names, in the form of simple
class names, i.e, " +
+ "for `set abc=xyz`, the value will be `SetCommand`. For those
auxiliary plans, such as " +
+ "`switch databases`, `set properties`, or `create temporary view`
e.t.c, " +
+ "which are used for setup evaluating environments for analyzing actual
queries, " +
+ "we can use this config to exclude them and let them take effect. " +
+ s"See also ${OPERATION_PLAN_ONLY_MODE.key}.")
+ .version("1.5.0")
+ .stringConf
+ .toSequence()
+ .createWithDefault(Seq("ResetCommand", "SetCommand",
"SetNamespaceCommand", "UseStatement"))
Review comment:
@iodone, the point above doesn't make sense. The engine side couldn't do
any mapping for the user-defined options.
--
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]
