You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by "Sunwoo-Shin (via GitHub)" <gi...@apache.org> on 2026/05/08 03:41:52 UTC
[I] [Bug] `kyuubi.server.info.provider` set in `kyuubi-defaults.conf` is silently ignored [kyuubi]
Sunwoo-Shin opened a new issue, #7435:
URL: https://github.com/apache/kyuubi/issues/7435
### 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/kyuubi/issues?q=is%3Aissue) and found no similar issues.
### Describe the bug
`kyuubi.server.info.provider` set in `kyuubi-defaults.conf` is silently ignored — `KyuubiSessionImpl#getInfo` always behaves as if the value were the default `ENGINE`; every `TGetInfo` call therefore waits for an engine to launch and is forwarded to it.
### Root cause
`SERVER_INFO_PROVIDER` is marked `.serverOnly`. `KyuubiSessionManager#openSession` builds the per-session conf via `getConf.getUserDefaults(user)`, which **unsets every `serverOnly` entry** before returning. As a result, `sessionConf.get(SERVER_INFO_PROVIDER)` in `KyuubiSessionImpl#getInfo` always returns the hard-coded default `"ENGINE"`, regardless of `kyuubi-defaults.conf`.
It only appears to work in tests because they pass the value as a per-session overlay (`withSessionConf(...)`), which is re-applied after `getUserDefaults`.
### Steps to reproduce
1. Set `kyuubi.server.info.provider=SERVER` in `kyuubi-defaults.conf`.
2. Connect via JDBC/beeline without overriding it per-session.
3. A `GetInfo` call still waits for an engine and the call is forwarded to it, instead of the server returning its own info directly.
### Expected vs Actual
- Expected: server-level `SERVER` setting takes effect; `getInfo` returns Kyuubi server info immediately.
- Actual: server-level setting is dropped; `getInfo` always takes the `ENGINE` branch.
### Suggested fix
Read `SERVER_INFO_PROVIDER` from the server-level conf, while still letting an explicit per-session override win:
```scala
val provider = sessionConf.getOption(SERVER_INFO_PROVIDER.key)
.getOrElse(sessionManager.getConf.get(SERVER_INFO_PROVIDER))
```
Making the entry non-`serverOnly` would also fix it, but would change the semantics for all other usages and would let clients flip the mode, so the targeted fix is preferred.
### Affects Version(s)
1.11.1
### Kyuubi Server Log Output
```logtalk
N/A — no error is logged. The bug is a silent fallback to the default value.
```
### Kyuubi Engine Log Output
```logtalk
N/A
```
### Kyuubi Server Configurations
```yaml
kyuubi.server.info.provider=SERVER
```
### Kyuubi Engine Configurations
```yaml
# N/A — engine-side config does not affect this bug.
```
### Additional context
_No response_
### Are you willing to submit PR?
- [x] Yes. I would be willing to submit a PR with guidance from the Kyuubi community to fix.
- [ ] No. I cannot submit a PR at this time.
--
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.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
Re: [I] [Bug] `kyuubi.server.info.provider` set in `kyuubi-defaults.conf` is silently ignored [kyuubi]
Posted by "Chaho12 (via GitHub)" <gi...@apache.org>.
Chaho12 commented on issue #7435:
URL: https://github.com/apache/kyuubi/issues/7435#issuecomment-4561349265
Can anyone provide feedback for this issue?
--
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
Re: [I] [Bug] `kyuubi.server.info.provider` set in `kyuubi-defaults.conf` is silently ignored [kyuubi]
Posted by "Chaho12 (via GitHub)" <gi...@apache.org>.
Chaho12 commented on issue #7435:
URL: https://github.com/apache/kyuubi/issues/7435#issuecomment-4737884270
Yes it is a bug, so by default it should read from server, and could be override by session.
--
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
Re: [I] [Bug] `kyuubi.server.info.provider` set in `kyuubi-defaults.conf` is silently ignored [kyuubi]
Posted by "ruanwenjun (via GitHub)" <gi...@apache.org>.
ruanwenjun commented on issue #7435:
URL: https://github.com/apache/kyuubi/issues/7435#issuecomment-4737829284
Seems a bug(I only see the code haven't tested this case), for all `server only` config we should get from kyuubi server conf rather then session config.
--
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
Re: [I] [Bug] `kyuubi.server.info.provider` set in `kyuubi-defaults.conf` is silently ignored [kyuubi]
Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 closed issue #7435: [Bug] `kyuubi.server.info.provider` set in `kyuubi-defaults.conf` is silently ignored
URL: https://github.com/apache/kyuubi/issues/7435
--
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