This is an automated email from the ASF dual-hosted git repository.
yashmayya pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git
The following commit(s) were added to refs/heads/master by this push:
new e084b5defa1 Fix Pinot JDBC driver metadata version retrieval (#17275)
e084b5defa1 is described below
commit e084b5defa1857b03fa1b4ddd5ee5a7db8e40ca1
Author: keriharris <[email protected]>
AuthorDate: Wed Nov 26 22:50:51 2025 +0100
Fix Pinot JDBC driver metadata version retrieval (#17275)
---
.../main/java/org/apache/pinot/client/PinotConnectionMetaData.java | 4 ++--
.../src/test/java/org/apache/pinot/client/PinotConnectionTest.java | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git
a/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotConnectionMetaData.java
b/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotConnectionMetaData.java
index 94e680f9cac..8dcf5678d5d 100644
---
a/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotConnectionMetaData.java
+++
b/pinot-clients/pinot-jdbc-client/src/main/java/org/apache/pinot/client/PinotConnectionMetaData.java
@@ -87,12 +87,12 @@ public class PinotConnectionMetaData extends
AbstractBaseConnectionMetaData {
@Override
public int getDriverMajorVersion() {
- return Integer.parseInt(DRIVER_VERSION.split(".")[0]);
+ return Integer.parseInt(DRIVER_VERSION.split("\\.")[0]);
}
@Override
public int getDriverMinorVersion() {
- return Integer.parseInt(DRIVER_VERSION.split(".")[1]);
+ return Integer.parseInt(DRIVER_VERSION.split("\\.")[1]);
}
@Override
diff --git
a/pinot-clients/pinot-jdbc-client/src/test/java/org/apache/pinot/client/PinotConnectionTest.java
b/pinot-clients/pinot-jdbc-client/src/test/java/org/apache/pinot/client/PinotConnectionTest.java
index 11112d2e341..9ced60999bd 100644
---
a/pinot-clients/pinot-jdbc-client/src/test/java/org/apache/pinot/client/PinotConnectionTest.java
+++
b/pinot-clients/pinot-jdbc-client/src/test/java/org/apache/pinot/client/PinotConnectionTest.java
@@ -61,6 +61,8 @@ public class PinotConnectionTest {
Assert.assertTrue(metaData.getDatabaseMajorVersion() > 0);
Assert.assertTrue(metaData.getDatabaseMinorVersion() >= 0);
Assert.assertEquals(metaData.getDatabaseProductName(), "APACHE_PINOT");
+ Assert.assertTrue(metaData.getDriverMajorVersion() > 0);
+ Assert.assertTrue(metaData.getDriverMinorVersion() >= 0);
Assert.assertEquals(metaData.getDriverName(), "APACHE_PINOT_DRIVER");
Assert.assertNotNull(metaData.getConnection());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
