Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[TASK][EASY] JDBC engine supports loading Jdbc Dialect from session conf #6316

Closed
3 of 4 tasks
cxzl25 opened this issue Apr 17, 2024 · 1 comment
Closed
3 of 4 tasks

[TASK][EASY] JDBC engine supports loading Jdbc Dialect from session conf #6316

cxzl25 opened this issue Apr 17, 2024 · 1 comment
Assignees
@lsm1

Comments

@cxzl25
Copy link
Contributor

cxzl25 commented Apr 17, 2024

Code of Conduct

Search before asking

  • I have searched in the issues and found no similar issues.

What would you like to be improved?

After #6182, we can pass kyuubi.engine.jdbc.connection.url through session conf, but JDBC Engine currently does not implement loading the corresponding jdbc dialect according to the corresponding url through session conf.

How should we improve?

No response

Are you willing to submit PR?

  • Yes. I would be willing to submit a PR with guidance from the Kyuubi community to improve.
  • No. I cannot submit a PR at this time.
@cxzl25 cxzl25 changed the title [Improvement] JDBC engine supports loading Jdbc Dialect from session conf [TASK][EASY][Improvement] JDBC engine supports loading Jdbc Dialect from session conf Apr 17, 2024
@cxzl25 cxzl25 mentioned this issue Apr 17, 2024
Closed
4 tasks
@pan3793 pan3793 changed the title [TASK][EASY][Improvement] JDBC engine supports loading Jdbc Dialect from session conf [TASK][EASY] JDBC engine supports loading Jdbc Dialect from session conf Apr 18, 2024
@lsm1
Copy link
Contributor

lsm1 commented Apr 18, 2024

please assign to me

@cxzl25 cxzl25 closed this as completed in 8b2bcfa Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lsm1 lsm1

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[KYUUBI #6316] Support switching jdbc dialect based on session conf lsm1/kyuubi
2 participants
@cxzl25 @lsm1