wangzhigang1999 commented on code in PR #7121:
URL: https://github.com/apache/kyuubi/pull/7121#discussion_r2192285359
##########
kyuubi-common/src/main/scala/org/apache/kyuubi/config/KyuubiConf.scala:
##########
@@ -2212,6 +2212,22 @@ object KyuubiConf {
.checkValue(_ >= 1000, "must >= 1s if set")
.createOptional
+ val OPERATION_TIMEOUT_POOL_SIZE: ConfigEntry[Int] =
+ buildConf("kyuubi.operation.timeout.pool.size")
+ .doc("Number of threads in the per-OperationManager timeout scheduler
used for " +
+ "operation-level timeout monitoring.")
+ .version("1.11.0")
+ .intConf
+ .createWithDefault(8)
+
+ val OPERATION_TIMEOUT_POOL_KEEPALIVE_TIME: ConfigEntry[Long] =
+ buildConf("kyuubi.operation.timeout.pool.keepalive.time")
+ .doc(
+ "Keep-alive time for idle threads in the per-OperationManager timeout
scheduler.")
Review Comment:
Each tweak is small, but together they make the code easier to read and
maintain. Thanks for catching them.
--
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]
