New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Improvement] Provide kyuubiServerPrincipal as alias for principal in JDBC parameter #3887
Closed
2 of 5 tasks
Labels
Comments
I am interested in this task. Please assign it to me. |
Thanks @lightning-L |
lightning-L
added a commit
to lightning-L/kyuubi
that referenced
this issue
Dec 8, 2022
…ipal in JDBC parameter
lightning-L
added a commit
to lightning-L/kyuubi
that referenced
this issue
Dec 8, 2022
…ipal in JDBC parameter
3 tasks
lightning-L
added a commit
to lightning-L/kyuubi
that referenced
this issue
Apr 24, 2023
pan3793
pushed a commit
that referenced
this issue
Apr 24, 2023
…mands ### _Why are the changes needed?_ ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [ ] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request Closes #4764 from lightning-L/kyuubi-3887-followup. Closes #3887 42a1232 [Tianlin Liao] [KYUUBI #3887][FOLLOWUP] fix kyuubiServerPrincipal logic in KyuubiCommands Authored-by: Tianlin Liao <tiliao@ebay.com> Signed-off-by: Cheng Pan <chengpan@apache.org>
pan3793
pushed a commit
that referenced
this issue
Apr 24, 2023
…mands ### _Why are the changes needed?_ ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [ ] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request Closes #4764 from lightning-L/kyuubi-3887-followup. Closes #3887 42a1232 [Tianlin Liao] [KYUUBI #3887][FOLLOWUP] fix kyuubiServerPrincipal logic in KyuubiCommands Authored-by: Tianlin Liao <tiliao@ebay.com> Signed-off-by: Cheng Pan <chengpan@apache.org> (cherry picked from commit b0d07f7) Signed-off-by: Cheng Pan <chengpan@apache.org>
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?
Kyuubi JDBC Driver introduced new parameters
kyuubiClientPrincipal
andkyuubiClientKeytab
in #3023, and it had a parameterprincipal
which is inherited from Hive JDBC Driver, it's quite confusing, so we'd better to add an aliaskyuubiServerPrincipal
forprincipal
to make it clear and consistent.How should we improve?
Provide
kyuubiServerPrincipal
as alias forprincipal
in JDBC parameterAre you willing to submit PR?
The text was updated successfully, but these errors were encountered: