[GitHub] [incubator-kyuubi] Yikf commented on a diff in pull request #3524: [KYUUBI #3512] Use dedicated ExecutionContext for EventBus async execution

[GitBox <gi...@apache.org>]

Yikf commented on code in PR #3524:
URL: https://github.com/apache/incubator-kyuubi/pull/3524#discussion_r974909339


##########
docs/deployment/settings.md:
##########
@@ -132,6 +132,15 @@ You can configure the Kyuubi properties in `$KYUUBI_HOME/conf/kyuubi-defaults.co
 # Details in https://kyuubi.apache.org/docs/latest/deployment/settings.html
 ```
 
+### Async
+
+Key | Default | Meaning | Type | Since
+--- | --- | --- | --- | ---
+kyuubi.async.event.handler.pool.keepalive.time|PT1M|Time(ms) that an idle async thread of the async event handler thread pool will wait for a new task to arrive before terminating|duration|1.7.0

Review Comment:
   It seems better to change to `kyuubi.event.async.handler.pool.keepalive.time`  as a separate event category



-- 
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-unsubscribe@kyuubi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org