Skip to content

Fix tvp issue with null value#412

Merged
AfsanehR-zz merged 5 commits intomicrosoft:devfrom
AfsanehR-zz:fix_TVP_issue_with_null_value
Jul 27, 2017
Merged

Fix tvp issue with null value#412
AfsanehR-zz merged 5 commits intomicrosoft:devfrom
AfsanehR-zz:fix_TVP_issue_with_null_value

Conversation

@AfsanehR-zz
Copy link
Copy Markdown
Contributor

@AfsanehR-zz AfsanehR-zz commented Jul 27, 2017

No description provided.

@AfsanehR-zz AfsanehR-zz requested a review from xiangyushawn July 27, 2017 17:33
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jul 27, 2017 •
edited
Loading

Codecov Report

Merging #412 into dev will decrease coverage by 0.32%.
The diff coverage is 50%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev     #412      +/-   ##
============================================
- Coverage     46.04%   45.71%   -0.33%     
+ Complexity     2203     2187      -16     
============================================
  Files           108      107       -1     
  Lines         25210    25206       -4     
  Branches       4164     4164              
============================================
- Hits          11608    11523      -85     
- Misses        11674    11752      +78     
- Partials       1928     1931       +3
Flag Coverage Δ Complexity Δ
#JDBC41 45.71% <50%> (-0.06%) 2187 <0> (ø)
#JDBC42 ? ?
Impacted Files Coverage Δ Complexity Δ
...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java 55.11% <50%> (-0.7%) 0 <0> (ø)
...t/sqlserver/jdbc/SQLServerPreparedStatement42.java 0% <0%> (-40%) 0% <0%> (-2%)
...erver/jdbc/SQLServerPreparedStatement42Helper.java 0% <0%> (-17.86%) 0% <0%> (-1%)
...ava/com/microsoft/sqlserver/jdbc/ScrollWindow.java 68.29% <0%> (-7.32%) 18% <0%> (-1%)
...microsoft/sqlserver/jdbc/SQLServerResultSet42.java 0% <0%> (-4.55%) 0% <0%> (-1%)
...om/microsoft/sqlserver/jdbc/ReaderInputStream.java 41.57% <0%> (-3.38%) 14% <0%> (-2%)
...t/sqlserver/jdbc/SQLServerCallableStatement42.java 0% <0%> (-2.64%) 0% <0%> (-1%)
...c/main/java/com/microsoft/sqlserver/jdbc/Util.java 59.17% <0%> (-1.52%) 90% <0%> (-2%)
...n/java/com/microsoft/sqlserver/jdbc/Parameter.java 57.51% <0%> (-1.29%) 55% <0%> (-1%)
... 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 c2e50a5...d428412. Read the comment docs.

@AfsanehR-zz AfsanehR-zz merged commit cb16472 into microsoft:dev Jul 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@xiangyushawn xiangyushawn xiangyushawn approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@AfsanehR-zz @codecov-io @xiangyushawn @lilgreenbird @msftclas @ajlam