apache / apache/kyuubi

[Bug] Engine space should be ignore the version change

Open
#2,734 6 comments 0 reactions 0 assignees View on GitHub
kind:bug priority:major
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.

### Describe the bug

We use `${serverSpace}_${KYUUBI_VERSION}_${shareLevel}_${engineType}` as the parent path of the engine space, but during hot upgrade kyuubi cluster with different version, we will have two version of engine space. As a result, a user share level engine can be created again since there are two engine space at that monent.

```
Client A -> Kyuubi Instance 1.5.0 -> engine A
Client A -> Kyuubi Instance 1.5.1 -> engine B
```

### Affects Version(s)

master/1.5

### Kyuubi Server Log Output

_No response_

### Kyuubi Engine Log Output

_No response_

### Kyuubi Server Configurations

_No response_

### Kyuubi Engine Configurations

_No response_

### Additional context

_No response_

### Are you willing to submit PR?

- [ ] Yes I am willing to submit a PR!

Contributor guide

Open the contributing guide

Research direction

Start by tracing where the engine-space parent path is built from `${serverSpace}_${KYUUBI_VERSION}_${shareLevel}_${engineType}` and review how hot upgrades create engine spaces across Kyuubi versions. Reproduce the 1.5.0/1.5.1 scenario, then verify that a shared engine is not created twice solely because the server versions differ.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
distributed-systems
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.