Skip to content

Conversation

@Eng-Fouad
Copy link
Contributor

Changelog:

[12.9.0] Preview Release

Added

Changed

Fixed issues

[12.10.0] Stable Release

Added

Changed

Fixed issues

@quarkus-bot quarkus-bot bot added area/dependencies Pull requests that update a dependency file area/jdbc Issues related to the JDBC extensions labels May 27, 2025
@Eng-Fouad
Copy link
Contributor Author

I think we need to register com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic for reflection in MsSQLReflections, but I am not sure. Are there test cases for MSSQL?

@gsmet
Copy link
Member

gsmet commented May 27, 2025

@Eng-Fouad
Copy link
Contributor Author

Eng-Fouad commented May 27, 2025

@gsmet Native Tests - Data 1 failed, but not sure if it is related to ConfigurableRetryLogic.

@Eng-Fouad
Copy link
Contributor Author

I tried mssql-jdbc 12.10.0 on native, and I got this exception:

Caused by: com.microsoft.sqlserver.jdbc.SQLServerException: Unable to locate specified class: com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic
	at com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic.getCurrentClassPath(ConfigurableRetryLogic.java:327)
	at com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic.readFromFile(ConfigurableRetryLogic.java:339)
	at com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic.setUpStatementRules(ConfigurableRetryLogic.java:207)
	at com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic.<init>(ConfigurableRetryLogic.java:89)
	at com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic.getInstance(ConfigurableRetryLogic.java:106)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connectInternal(SQLServerConnection.java:2600)
	at com.microsoft.sqlserver.jdbc.SQLServerConnection.connect(SQLServerConnection.java:2207)
	at com.microsoft.sqlserver.jdbc.SQLServerDriver.connect(SQLServerDriver.java:1320)
	at io.agroal.pool.ConnectionFactory.createConnection(ConnectionFactory.java:225)
	at io.agroal.pool.ConnectionPool$CreateConnectionTask.call(ConnectionPool.java:585)
	at io.agroal.pool.ConnectionPool$CreateConnectionTask.call(ConnectionPool.java:566)
	at [email protected]/java.util.concurrent.FutureTask.run(FutureTask.java:328)
	at io.agroal.pool.util.PriorityScheduledExecutor.beforeExecute(PriorityScheduledExecutor.java:75)
	at [email protected]/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1093)
	at [email protected]/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:619)
	... 4 more
Caused by: java.lang.ClassNotFoundException: com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic
	at org.graalvm.nativeimage.builder/com.oracle.svm.core.hub.ClassForNameSupport.forName(ClassForNameSupport.java:215)
	at org.graalvm.nativeimage.builder/com.oracle.svm.core.hub.ClassForNameSupport.forName(ClassForNameSupport.java:183)
	at [email protected]/java.lang.Class.forName(DynamicHub.java:1509)
	at [email protected]/java.lang.Class.forName(DynamicHub.java:1468)
	at [email protected]/java.lang.Class.forName(DynamicHub.java:1461)
	at com.microsoft.sqlserver.jdbc.ConfigurableRetryLogic.getCurrentClassPath(ConfigurableRetryLogic.java:294)
	... 18 more

https://github.com/microsoft/mssql-jdbc/blob/a0ea23a0de0ec314b9836664cd158c4a795174dc/src/main/java/com/microsoft/sqlserver/jdbc/ConfigurableRetryLogic.java#L294

I will update MsSQLReflections accordingly.

@Eng-Fouad Eng-Fouad force-pushed the patch-1 branch 3 times, most recently from adef1b6 to b16c0bf Compare May 28, 2025 13:59
@gastaldi gastaldi added the triage/waiting-for-ci Ready to merge when CI successfully finishes label May 28, 2025
@Eng-Fouad Eng-Fouad force-pushed the patch-1 branch 2 times, most recently from d620c7d to e703b0c Compare May 29, 2025 08:31
@quarkus-bot
Copy link

quarkus-bot bot commented May 29, 2025

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit 986d05d.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

@gsmet gsmet merged commit cfdec81 into quarkusio:main May 30, 2025
57 checks passed
@quarkus-bot quarkus-bot bot added this to the 3.24 - main milestone May 30, 2025
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label May 30, 2025
@Eng-Fouad Eng-Fouad deleted the patch-1 branch May 30, 2025 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dependencies Pull requests that update a dependency file area/jdbc Issues related to the JDBC extensions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants