[Improvement] Make KyuubiThriftBinaryFrontendService handler thread pool daemon
- Dominant language
- Scala
- Stars
- 2.4k
- Forks
- 1k
- PR merge metrics
- No merged PRs in 30d
Description
### Code of Conduct
- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
### Search before asking
- [X] I have searched in the [issues](https://github.com/apache/incubator-kyuubi/issues?q=is%3Aissue) and found no similar issues.
### What would you like to be improved?
Since the KyuubiThriftBinaryFrontendService thread is not a daemon, it will wait for the KyuubiThriftBinaryFrontendService handler thread to finish or time out, so we can make KyuubiThriftBinaryFrontendService handler thread daemon.
FYI: org.apache.thrift.server.TThreadPoolServer#serve:
### How should we improve?
_No response_
### Are you willing to submit PR?
- [X] Yes I am willing to submit a PR!
Contributor guide
Research direction
Start with KyuubiThriftBinaryFrontendService and the referenced org.apache.thrift.server.TThreadPoolServer#serve entry point to trace how handler threads are created. Confirm the service handler thread is daemonized and verify that shutdown no longer waits for it to finish or time out; no source file or test path is named in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scala
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100