Skip to content

The Clickhouse JDBC driver doesn't report driver version #2245

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

Closed
rogerbj opened this issue Mar 25, 2025 · 3 comments · Fixed by #2296
Closed

The Clickhouse JDBC driver doesn't report driver version #2245

rogerbj opened this issue Mar 25, 2025 · 3 comments · Fixed by #2296
Assignees
Labels
Milestone

Comments

@rogerbj
Copy link

rogerbj commented Mar 25, 2025

Connected to Clickhouse with this JDBC driver:

<dependency>
   <groupId>com.clickhouse</groupId>
   <artifactId>clickhouse-jdbc</artifactId>
   <version>0.8.2</version>
   <type></type>
</dependency>

Calling DatabaseMetaData.getDriverVersion() returns unknown.
Expected: It should return the actual version which in this case is 0.8.2.

@rogerbj rogerbj added the bug label Mar 25, 2025
@Paultagoras Paultagoras added this to the 0.8.4 milestone Apr 8, 2025
@mzitnik mzitnik self-assigned this Apr 8, 2025
mzitnik added a commit that referenced this issue Apr 9, 2025
@mzitnik mzitnik mentioned this issue Apr 9, 2025
4 tasks
@mzitnik
Copy link
Contributor

mzitnik commented Apr 9, 2025

@rogerbj, where did you run the DatabaseMetaData? GetDriverVersion (): Is it from a simple app or some Spring application?
I have run it from an external app, and looks fine

@rogerbj
Copy link
Author

rogerbj commented Apr 10, 2025

Mark, This is when using our DbVisualizer app.

@mzitnik
Copy link
Contributor

mzitnik commented Apr 10, 2025

@rogerbj I added a fallback in loading the property. You can test against #2296 or just wait until 0.8.4, which will be published at the end of next week.

mzitnik added a commit that referenced this issue Apr 10, 2025
* Adding issue #2245 test

* Adding to readVersionFromResource the ability to load from a different classloader for those cases when the thread is running in a different classloader
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants