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

Release | Merge dev to master for v8.1.0-preview #1161

Merged
merged 18 commits into from
Oct 16, 2019
Merged

Release | Merge dev to master for v8.1.0-preview #1161

merged 18 commits into from
Oct 16, 2019

Conversation

ulvii
Copy link
Contributor

@ulvii ulvii commented Oct 16, 2019

No description provided.

rene-ye and others added 18 commits August 2, 2019 17:39
@rene-ye
Merge pull request #1124 from microsoft/master
80c57a1 
Merge pull request #1123 from microsoft/dev
@ulvii
Release | Update SNAPSHOT for 7.5.0-preview release (#1129)
8e4c50b 
* Release | Update driver version

* Release| Add pom and gradle files
@snuyanzin @ulvii
Fix | Remove extra spaces in SQLServerDatabaseMetaData.getNumericFunc…
1f27d64 
…tions()/SQLServerDatabaseMetaData.getStringFunctions() return values (#1117)
@harawata @ulvii
Tests | Improve assertions in ComparisonUtil (#1100)
27ceebf 
* ComparisonUtil : replace assertTrue with assertEquals

Failure message should include the actual/expected values and that is what assertEquals does.

* Corrected BigDecimal comparison

Scale difference should be ignored.
@lilgreenbird @ulvii
Fix | Add list of trusted endpoints for AKV (#1130)
8a5fca2 
* added list of trusted endpoints

* rename
@ulvii
Update issue templates (#1148)
557ac93
@lilgreenbird
Fix getImportedKeys() returning duplicate rows if multiple FKs have t…
40fb4fc 
…he same name (#1092)

* fixed for github #1091 dup rows
@peterbae
Fix STAsBinary returning null for a single point (#1074)
7dd953e 
* populate wkb for point

* fix stasbinary issue

* change variable name

* apply same variable name change
@rene-ye
Performance | Improved performance of column name string lookups (#1066)
b424b77 
* Fix | Made column name lookup more performant

* Fix | Spacing

* Fix | Changed arraylist to maps

* Fix | Add newline

* Add | Caching for previously retrieved columns

* Fix | Formatting

* Fix | Variable issue

* Fix | null order

* Fix | part 2

* Revert | Integer changes

* Fix | Trim retrieved column name

* Fix | Clear column names whenever ResultSet cursor is moved

* Revert "Fix | Clear column names whenever ResultSet cursor is moved"

This reverts commit db42d44.
@ulvii
Add | Implement hashCode() and equals() APIs for SQLServerDataTable a…
2deb6fa 
…nd SQLServerDataColumn (#1146)

* Add | Add hashCode()/equals() methods to SQLServerDataColumn and SQLServerDataTable
@peterbae
Fix issue with truststore password being removed too early for XA tra…
8e61500 
…nsaction (#1133)

Fix | Fix issue with truststore password being removed too early for XA transaction
@peterbae @ulvii
Fix | SQLServerDatabaseMetada.getColumns not escaping wildcard (#1138)
aed8307
@ulvii
Feature | Introduce JAVA 13 Support (#1151)
86914b9 
* JDK 13 | Remove jre12 from pom file

* JDK 13 | Update Gradle build file

* JDK 13 | Update Azure-pipelines
@ulvii
Test | Updated SQL Server from 2008R2 to 2012 in CI (#1153)
e65cb56
@ulvii
Performance | Disabled pattern matching when using CallableStatements…
94beeac 
… and SQLServerParameterMetaData (#1149)
@jonfreedman @ulvii
Fix | Added ISQLServerBulkData to remove implementation details from …
02da869 
…ISQLServerBulkRecord (#1099)
@rene-ye @ulvii
Feature | Added support for Always Encrypted with Secure Enclaves (#1155
276d2ca 
)
@ulvii
Release | 8.1.0-preview release changes (#1158)
a8f5fee
@codecov-io
Copy link

codecov-io commented Oct 16, 2019 •
edited
Loading

Codecov Report

❗ No coverage uploaded for pull request base (master@7f28e9c). Click here to learn what that means.
The diff coverage is 25.61%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #1161   +/-   ##
=========================================
  Coverage          ?   53.87%           
  Complexity        ?     3390           
=========================================
  Files             ?      126           
  Lines             ?    29321           
  Branches          ?     4849           
=========================================
  Hits              ?    15796           
  Misses            ?    11203           
  Partials          ?     2322
Impacted Files Coverage Δ Complexity Δ
...crosoft/sqlserver/jdbc/KeyStoreProviderCommon.java 63.01% <ø> (ø) 9 <0> (?)
...dbc/SQLServerColumnEncryptionKeyStoreProvider.java 100% <ø> (ø) 1 <0> (?)
...om/microsoft/sqlserver/jdbc/AuthenticationJNI.java 0% <ø> (ø) 0 <0> (?)
...om/microsoft/sqlserver/jdbc/SQLServerResource.java 100% <ø> (ø) 4 <0> (?)
...a/com/microsoft/sqlserver/jdbc/SQLJdbcVersion.java 0% <ø> (ø) 0 <0> (?)
...oft/sqlserver/jdbc/SQLServerParameterMetaData.java 68.13% <0%> (ø) 57 <0> (?)
...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java 40.33% <0%> (ø) 77 <0> (?)
...erverColumnEncryptionCertificateStoreProvider.java 31.03% <0%> (ø) 3 <0> (?)
...soft/sqlserver/jdbc/ISQLServerEnclaveProvider.java 0% <0%> (ø) 0 <0> (?)
...QLServerColumnEncryptionAzureKeyVaultProvider.java 0% <0%> (ø) 0 <0> (?)
... and 18 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7f28e9c...a8f5fee. Read the comment docs.

@ulvii ulvii merged commit e88884c into master Oct 16, 2019
ulvii added a commit that referenced this pull request Oct 16, 2019
@ulvii
Revert "Release | Merge dev to master for v8.1.0-preview (#1161)"
439c0d6 
This reverts commit e88884c.
@ulvii ulvii mentioned this pull request Oct 16, 2019
Merged
ulvii added a commit that referenced this pull request Oct 16, 2019
@ulvii
Revert "Release | Merge dev to master for v8.1.0-preview (#1161)" (#1163
060c657 
)

This reverts commit e88884c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@ulvii @codecov-io @rene-ye @snuyanzin @harawata @lilgreenbird @peterbae @jonfreedman