Skip to content

Removal of deprecated APIs and update travic script with jdk 9#577

Merged
AfsanehR-zz merged 5 commits intomicrosoft:JDBC4.3from
AfsanehR-zz:Java9
Dec 11, 2017
Merged

Removal of deprecated APIs and update travic script with jdk 9#577
AfsanehR-zz merged 5 commits intomicrosoft:JDBC4.3from
AfsanehR-zz:Java9

Conversation

@AfsanehR-zz
Copy link
Copy Markdown
Contributor

No description provided.

@codecov-io
Copy link
Copy Markdown

codecov-io commented Dec 7, 2017

Codecov Report

Merging #577 into JDBC4.3 will decrease coverage by 0.06%.
The diff coverage is 31.03%.

Impacted file tree graph

@@              Coverage Diff              @@
##             JDBC4.3     #577      +/-   ##
=============================================
- Coverage      46.52%   46.46%   -0.07%     
+ Complexity      2217     2213       -4     
=============================================
  Files            108      108              
  Lines          25363    25373      +10     
  Branches        4177     4177              
=============================================
- Hits           11800    11789      -11     
- Misses         11533    11560      +27     
+ Partials        2030     2024       -6
Flag Coverage Δ Complexity Δ
#JDBC41 46.4% <31.03%> (-0.04%) 2210 <0> (-1)
#JDBC42 46.3% <31.03%> (-0.07%) 2205 <0> (-7)
Impacted Files Coverage Δ Complexity Δ
...ava/com/microsoft/sqlserver/jdbc/FailOverInfo.java 52.63% <0%> (ø) 0 <0> (ø) ⬇️
.../com/microsoft/sqlserver/jdbc/SQLServerSQLXML.java 12.8% <0%> (-0.11%) 9 <0> (ø)
...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java 62.84% <0%> (-0.37%) 0 <0> (ø)
.../java/com/microsoft/sqlserver/jdbc/SqlVariant.java 88.33% <0%> (ø) 13 <0> (ø) ⬇️
...lserver/jdbc/SQLServerDataSourceObjectFactory.java 0% <0%> (ø) 0 <0> (ø) ⬇️
.../com/microsoft/sqlserver/jdbc/SQLServerDriver.java 77.22% <100%> (ø) 25 <0> (ø) ⬇️
.../microsoft/sqlserver/jdbc/SQLServerDataSource.java 46.56% <100%> (ø) 68 <0> (ø) ⬇️
...oft/sqlserver/jdbc/SQLServerBulkCSVFileRecord.java 45.92% <100%> (ø) 30 <0> (ø) ⬇️
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java 44.24% <50%> (-0.68%) 103 <0> (-2)
.../microsoft/sqlserver/jdbc/SQLServerConnection.java 45.47% <71.42%> (ø) 277 <0> (ø) ⬇️
... and 9 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 b1dbacc...47ddc43. Read the comment docs.

Copy link
Copy Markdown
Contributor

@xiangyushawn xiangyushawn left a comment

Choose a reason for hiding this comment

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

The PR looks good to me. just wondering why we use Integer.valueOf() sometimes and sometimes we use Integer.parseInt()?

@AfsanehR-zz
Copy link
Copy Markdown
Contributor Author

@v-xiangs good point. The difference between those two is that Integer.parseInt() return primitive int and valueOf returns java.lang.Integer. I mostly used Integer.parseInt(String) except for the line that we need to convert Int to string. for example here: using Integer.valueOf() allows us to call toString() method on the Object.

@AfsanehR-zz AfsanehR-zz changed the title Removal of deprecated APIs Removal of deprecated APIs and update travic script with jdk 9 Dec 8, 2017
@AfsanehR-zz AfsanehR-zz merged commit 7a2bbef into microsoft:JDBC4.3 Dec 11, 2017
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