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

Re-introduce Retry logic for prepared statement caching - JRE 7 branch #620

Merged
merged 11 commits into from
Feb 15, 2018

Conversation

cheenamalhotra
Copy link
Member

No description provided.

@cheenamalhotra cheenamalhotra added this to the 6.4.0 milestone Feb 1, 2018
@codecov-io
Copy link

codecov-io commented Feb 1, 2018

Codecov Report

Merging #620 into RTW-V6.4.0.jre7 will decrease coverage by 0.04%.
The diff coverage is 67.88%.

Impacted file tree graph

@@                  Coverage Diff                  @@
##             RTW-V6.4.0.jre7     #620      +/-   ##
=====================================================
- Coverage              46.47%   46.43%   -0.05%     
- Complexity              2211     2215       +4     
=====================================================
  Files                    108      108              
  Lines                  25343    25343              
  Branches                4178     4181       +3     
=====================================================
- Hits                   11779    11768      -11     
- Misses                 11542    11544       +2     
- Partials                2022     2031       +9
Flag Coverage Δ Complexity Δ
#JDBC41 46.28% <67.88%> (+0.05%) 2205 <15> (+4) ⬆️
#JDBC42 46.3% <67.88%> (-0.05%) 2211 <15> (+4)
Impacted Files Coverage Δ Complexity Δ
...t/sqlserver/jdbc/SQLServerConnectionPoolProxy.java 14.65% <ø> (-0.45%) 14 <0> (ø)
.../microsoft/sqlserver/jdbc/SQLServerDataSource.java 45.5% <0%> (-0.49%) 66 <0> (ø)
...m/microsoft/sqlserver/jdbc/SQLServerStatement.java 59.47% <100%> (-0.19%) 131 <0> (-2)
.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java 77% <100%> (ø) 25 <0> (ø) ⬇️
...oft/sqlserver/jdbc/SQLServerPreparedStatement.java 50.4% <64.55%> (-0.96%) 155 <2> (-5)
.../microsoft/sqlserver/jdbc/SQLServerConnection.java 45.96% <87.5%> (+0.15%) 287 <13> (+9) ⬆️
...om/microsoft/sqlserver/jdbc/SimpleInputStream.java 51.11% <0%> (-1.49%) 11% <0%> (-1%)
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java 43.34% <0%> (-0.91%) 104% <0%> (+1%)
...ooglecode/concurrentlinkedhashmap/LinkedDeque.java 27.69% <0%> (-0.77%) 19% <0%> (-1%)
...c/main/java/com/microsoft/sqlserver/jdbc/Util.java 60.88% <0%> (-0.45%) 87% <0%> (-1%)
... and 10 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 6a76058...8270a9a. Read the comment docs.

@cheenamalhotra cheenamalhotra added this to Under Peer Review in MSSQL JDBC Feb 9, 2018
@cheenamalhotra cheenamalhotra merged commit a46e18a into microsoft:RTW-V6.4.0.jre7 Feb 15, 2018
MSSQL JDBC automation moved this from Under Peer Review to Done Feb 15, 2018
@cheenamalhotra cheenamalhotra moved this from Closed Issues to Closed/Merged PRs in MSSQL JDBC Mar 3, 2018
@cheenamalhotra cheenamalhotra removed this from Closed/Merged PRs in MSSQL JDBC Mar 3, 2018
@lilgreenbird lilgreenbird added this to Closed/Merged PRs in MSSQL JDBC Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
MSSQL JDBC
  
Closed/Merged PRs
Development

Successfully merging this pull request may close these issues.

None yet

3 participants