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

Usual changes for preview release 6.5.2 #692

Merged
merged 4 commits into from
May 4, 2018

Conversation

peterbae
Copy link
Contributor

@peterbae peterbae commented May 4, 2018

Also updated Gradle / adal4j dependencies as well.

@peterbae peterbae changed the title Usual changes for preview release, includes Gradle update. Usual changes for preview release 6.5.2 May 4, 2018
CHANGELOG.md Outdated

### Changed
- Updated Maven dependencies for test suite [#676](https://github.com/Microsoft/mssql-jdbc/pull/676)
- Updated multiple Maven dependency and plugin versions [#688](https://github.com/Microsoft/mssql-jdbc/pull/688)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's squash "Updated Maven Dependencies" into one line and move the 2 PRs listed in 'Added' here. They're udpates/modifications not something new.

CHANGELOG.md Outdated
### Fixed Issues
- Fixed an issue where ResultSetMetaData instances created by a ResultSet that has been closed were not persisting [#685](https://github.com/Microsoft/mssql-jdbc/pull/685)
- Fixed an issue with PreparedStatement.setBigDecimal when no scale is passed [#684](https://github.com/Microsoft/mssql-jdbc/pull/684)
- Fixed an issue with Statement.setQueryTimeout not throwing an exception when no network is available [#674](https://github.com/Microsoft/mssql-jdbc/pull/674)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change description for this one. #674 adds new connection property, should be moved to Added Section

ulvii
ulvii previously approved these changes May 4, 2018
CHANGELOG.md Outdated
- Fixed an issue where ResultSetMetaData instances created by a ResultSet that has been closed were not persisting [#685](https://github.com/Microsoft/mssql-jdbc/pull/685)
- Fixed an issue with PreparedStatement.setBigDecimal when no scale is passed [#684](https://github.com/Microsoft/mssql-jdbc/pull/684)
- Fixed an issue with Statement.setQueryTimeout not throwing an exception when no network is available [#674](https://github.com/Microsoft/mssql-jdbc/pull/674)
- Fixed an issue with SQLServerDatabaseMetaData having performance degradation in 6.4.0 [#677](https://github.com/Microsoft/mssql-jdbc/pull/677)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change description to PR title "Improved performance degradation while maintaining JDBC compliance with results from sp_fkeys". Move this one to top of the Fixed Issues List.

@codecov-io
Copy link

codecov-io commented May 4, 2018

Codecov Report

Merging #692 into dev will decrease coverage by 0.11%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev     #692      +/-   ##
============================================
- Coverage     48.15%   48.03%   -0.12%     
+ Complexity     2585     2579       -6     
============================================
  Files           114      114              
  Lines         26578    26578              
  Branches       4453     4453              
============================================
- Hits          12798    12768      -30     
- Misses        11638    11668      +30     
  Partials       2142     2142
Flag Coverage Δ Complexity Δ
#JDBC42 47.95% <ø> (-0.12%) 2576 <ø> (-3)
#JDBC43 47.95% <ø> (-0.1%) 2577 <ø> (-5)
Impacted Files Coverage Δ Complexity Δ
...a/com/microsoft/sqlserver/jdbc/SQLJdbcVersion.java 0% <ø> (ø) 0 <0> (ø) ⬇️
...om/microsoft/sqlserver/jdbc/ReaderInputStream.java 42.69% <0%> (-2.25%) 14% <0%> (-2%)
...n/java/com/microsoft/sqlserver/jdbc/Parameter.java 61.92% <0%> (-0.84%) 63% <0%> (ø)
...c/main/java/com/microsoft/sqlserver/jdbc/Util.java 60.91% <0%> (-0.66%) 88% <0%> (-1%)
...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java 54.8% <0%> (-0.47%) 0% <0%> (ø)
...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java 32.85% <0%> (-0.36%) 242% <0%> (-3%)
...oft/sqlserver/jdbc/SQLServerPreparedStatement.java 50.44% <0%> (-0.25%) 157% <0%> (ø)
...m/microsoft/sqlserver/jdbc/SQLServerStatement.java 59.44% <0%> (-0.1%) 134% <0%> (-1%)
...n/java/com/microsoft/sqlserver/jdbc/DataTypes.java 78.01% <0%> (+0.16%) 5% <0%> (+1%) ⬆️
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java 45.63% <0%> (+0.22%) 107% <0%> (-1%) ⬇️
... and 1 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 fc50887...73c6464. Read the comment docs.

@peterbae peterbae merged commit a16ef34 into microsoft:dev May 4, 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.

4 participants