[Improvement] connection limits need change by SessionConfAdvisor
- 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?
there are 3 parameters for limit connections:
```
kyuubi.server.limit.connections.per.user
kyuubi.server.limit.connections.per.ipaddress
kyuubi.server.limit.connections.per.user.ipaddress
```
I found these params can only be initialized by kyuubi-default.conf, which may not change by SessionConfAdvisor or another dynamic parameter way. However, devops need control the connection limit temporary sometimes, for passing big amount of user's connections or stopping some user's connection.
### How should we improve?
connection limits related params may need change by SessionConfAdvisor, or some other dynamic ways
### Are you willing to submit PR?
- [ ] Yes. I can submit a PR independently to improve.
- [ ] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
- [X] No. I cannot submit a PR at this time.
Contributor guide
Research direction
Start by tracing how kyuubi.server.limit.connections.per.user, per.ipaddress, and per.user.ipaddress are initialized and how SessionConfAdvisor handles dynamic settings. Determine the supported dynamic-update entry point and verify that operators can temporarily change each limit and that the connection behavior reflects those changes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- scala
- Domain
- api, backend
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 30/100