-
Notifications
You must be signed in to change notification settings - Fork 6
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
[DOP-13852] - upgrade MSSQL packages #254
Conversation
b847214
to
b3ed228
Compare
b3ed228
to
a48f51b
Compare
Кажется именно для 12.7 нет jre8 и jre11, есть только jre8-preview и jre11-preview: https://mvnrepository.com/artifact/com.microsoft.sqlserver/mssql-jdbc Да и в документации его тоже пока нет: https://learn.microsoft.com/en-us/sql/connect/jdbc/microsoft-jdbc-driver-for-sql-server-support-matrix?view=sql-server-ver16#java-and-jdbc-specification-support Давай пока остановимся на 12.6.1. |
3e2f4b9
to
8f12c3c
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #254 +/- ##
===========================================
+ Coverage 94.39% 94.40% +0.01%
===========================================
Files 204 204
Lines 7901 7905 +4
Branches 1402 1402
===========================================
+ Hits 7458 7463 +5
+ Misses 318 317 -1
Partials 125 125 ☔ View full report in Codecov by Sentry. |
8f12c3c
to
4ce1c6d
Compare
Change Summary
MSSQL
connection now uses Microsoft SQL Server JDBC driver12.7.0
, upgraded from12.2.0
, and supports passing custom versions:MSSQL.get_packages(java_version=..., package_version=...)
.Related issue number
Checklist
docs/changelog/next_release/<pull request or issue id>.<change type>.rst
file added describing change(see CONTRIBUTING.rst for details.)