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.

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

Merging dev to master for 6.5.3-preview Release #713

Merged
merged 49 commits into from
Jun 1, 2018
Merged

Merging dev to master for 6.5.3-preview Release #713

merged 49 commits into from
Jun 1, 2018

Conversation

cheenamalhotra
Copy link
Member

No description provided.

peterbae and others added 30 commits February 23, 2018 10:13
… old Constructor and added a new constructor with 1 param
…d by getColumns() (#695)

* Fix | getColumns() API, changed column name from SS_IS_COMPUTED to IS_AUTOINCREMENT per JDBC specs | issue #600

* Fix | getColumns() API, changed column name from SS_IS_COMPUTED to IS_GENERATEDCOLUMN per JDBC specs | issue #600
cheenamalhotra and others added 19 commits May 23, 2018 15:38
Adding Old Constructor back to AKV Provider
…API (#701)

* Resolving maven warnings

* Removing jreVersion property

Does not make sense now that we use final name in the build itself. Only used in 1 place, hard-coding java version for different builds as that's what it represents anyways.

* java warnings
resource bundle for error message strings in junit tests
Fix for Prepared Statement handle not found intermittent exception
* Feature | Added support for JDK 10 in both Maven and Gradle - builds jre10 jars for the driver, replacing jre9

* JDK 10 | Merge 42 classes to base classes to reduce class redundancy.

* JDK 10 | Attempt to run JDK 10 with Appveyor

* Remove unwanted space

* Updating Travis script to use JDK 10

* Testing without addons

* Update script for Jacoco report to build 43 profile

* Revert driver changes for 42 compliance - to be added in a separate PR

* Revert Test class changes for 42 compliance - to be done in a separate PR

* Reformatted code

* Add ID to jacoco plugin execution task
)

fix for automatic credential discarding

* update felix to 3.5.0

* Revised implementation

Decided to not dispose user created credentials at all.

* Updated flag set location
* changes for preview release

* requested changes

* jre version update changes
@cheenamalhotra cheenamalhotra added this to the 6.5.3 milestone Jun 1, 2018
@codecov-io
Copy link

codecov-io commented Jun 1, 2018

Codecov Report

Merging #713 into master will decrease coverage by 0.06%.
The diff coverage is 26.19%.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #713      +/-   ##
===========================================
- Coverage     48.17%   48.1%   -0.07%     
+ Complexity     2585    2582       -3     
===========================================
  Files           114     113       -1     
  Lines         26578   26553      -25     
  Branches       4453    4457       +4     
===========================================
- Hits          12803   12773      -30     
- Misses        11635   11648      +13     
+ Partials       2140    2132       -8
Flag Coverage Δ Complexity Δ
#JDBC42 47.98% <26.19%> (-0.11%) 2571 <10> (-12)
#JDBC43 48.04% <26.19%> (-0.08%) 2581 <10> (-2)
Impacted Files Coverage Δ Complexity Δ
...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java 32.26% <ø> (-0.16%) 58 <0> (ø)
...a/com/microsoft/sqlserver/jdbc/SQLJdbcVersion.java 0% <ø> (ø) 0 <0> (ø) ⬇️
.../microsoft/sqlserver/jdbc/SQLServerDataSource.java 45.19% <ø> (ø) 66 <0> (ø) ⬇️
...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java 52.61% <0%> (-0.19%) 240 <0> (ø)
...m/microsoft/sqlserver/jdbc/SQLServerStatement.java 59.36% <0%> (-0.19%) 135 <0> (ø)
...m/microsoft/sqlserver/jdbc/KerbAuthentication.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...m/microsoft/sqlserver/jdbc/KeyVaultCredential.java 0% <0%> (ø) 0 <0> (ø) ⬇️
...QLServerColumnEncryptionAzureKeyVaultProvider.java 0% <0%> (ø) 0 <0> (ø) ⬇️
.../microsoft/sqlserver/jdbc/SQLServerConnection.java 45.76% <31.25%> (+0.06%) 288 <0> (ø) ⬇️
...oft/sqlserver/jdbc/SQLServerPreparedStatement.java 50.61% <53.12%> (-0.08%) 160 <10> (+3)
... and 24 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 6815b08...c8e638d. Read the comment docs.

@rene-ye rene-ye self-requested a review June 1, 2018 20:23
@cheenamalhotra cheenamalhotra merged commit 405c704 into master Jun 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants