apache / apache/kyuubi

[Improvement] connection limits need change by SessionConfAdvisor

Open
#3,673 2 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.