[Improvement] Remove flink. prefix for open session configurations
#6156
Assignees
Comments
4 tasks
pan3793
pushed a commit
that referenced
this issue
Mar 12, 2024
# 🔍 Description ## Issue References 🔗 This pull request fixes #6156 ## Describe Your Solution 🔧 Remove `flink.` prefix for open flink session configurations. ## Types of changes 🔖 - [x] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [X] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6157 from wForget/KYUUBI-6156. Closes #6156 fc750dc [wforget] comment f613491 [wforget] Remove `flink.` prefix for create session configurations Authored-by: wforget <[email protected]> Signed-off-by: Cheng Pan <[email protected]> (cherry picked from commit 7562a97) Signed-off-by: Cheng Pan <[email protected]>
zhaohehuhu
pushed a commit
to zhaohehuhu/incubator-kyuubi
that referenced
this issue
Mar 21, 2024
…urations # 🔍 Description ## Issue References 🔗 This pull request fixes apache#6156 ## Describe Your Solution 🔧 Remove `flink.` prefix for open flink session configurations. ## Types of changes 🔖 - [x] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [X] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes apache#6157 from wForget/KYUUBI-6156. Closes apache#6156 fc750dc [wforget] comment f613491 [wforget] Remove `flink.` prefix for create session configurations Authored-by: wforget <[email protected]> Signed-off-by: Cheng Pan <[email protected]>
beryllw
pushed a commit
to beryllw/incubator-kyuubi
that referenced
this issue
Jun 7, 2024
…urations # 🔍 Description ## Issue References 🔗 This pull request fixes apache#6156 ## Describe Your Solution 🔧 Remove `flink.` prefix for open flink session configurations. ## Types of changes 🔖 - [x] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [X] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes apache#6157 from wForget/KYUUBI-6156. Closes apache#6156 fc750dc [wforget] comment f613491 [wforget] Remove `flink.` prefix for create session configurations Authored-by: wforget <[email protected]> Signed-off-by: Cheng Pan <[email protected]> (cherry picked from commit 7562a97)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Code of Conduct
Search before asking
What would you like to be improved?
When opening session in flink engine, we should remove
flink.prefix for configurations.How should we improve?
No response
Are you willing to submit PR?
The text was updated successfully, but these errors were encountered: