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

TVP and BulkCopy random data test for all data types with server cursor #319

Merged
merged 16 commits into from
Jun 27, 2017

Conversation

xiangyushawn
Copy link
Contributor

No description provided.

@codecov-io
Copy link

codecov-io commented May 30, 2017

Codecov Report

Merging #319 into RTW_6.2.0 will increase coverage by 2.92%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             RTW_6.2.0     #319      +/-   ##
===============================================
+ Coverage        37.17%   40.09%   +2.92%     
- Complexity        1670     1877     +207     
===============================================
  Files              103      107       +4     
  Lines            23663    24484     +821     
  Branches          3918     4037     +119     
===============================================
+ Hits              8796     9818    +1022     
+ Misses           13278    12822     -456     
- Partials          1589     1844     +255
Flag Coverage Δ Complexity Δ
#JDBC41 39.91% <ø> (+2.87%) 1870 <ø> (+206) ⬆️
#JDBC42 40% <ø> (+2.96%) 1873 <ø> (+213) ⬆️
Impacted Files Coverage Δ Complexity Δ
...om/microsoft/sqlserver/jdbc/SimpleInputStream.java 47.4% <0%> (-1.49%) 9% <0%> (-1%)
...om/microsoft/sqlserver/jdbc/ReaderInputStream.java 44.94% <0%> (-1.13%) 16% <0%> (-1%)
...n/java/com/microsoft/sqlserver/jdbc/tdsparser.java 68.46% <0%> (-0.91%) 0% <0%> (ø)
...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java 45.19% <0%> (-0.69%) 0% <0%> (ø)
...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java 26.86% <0%> (-0.38%) 184% <0%> (-3%)
...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java 29.01% <0%> (-0.12%) 46% <0%> (ø)
...m/microsoft/sqlserver/jdbc/SQLServerStatement.java 59.06% <0%> (-0.02%) 129% <0%> (ø)
...oft/sqlserver/jdbc/SQLServerParameterMetaData.java 25.52% <0%> (ø) 30% <0%> (ø) ⬇️
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java 30.11% <0%> (ø) 58% <0%> (+2%) ⬆️
...l/googlecode/concurrentlinkedhashmap/Weighers.java 8.51% <0%> (ø) 1% <0%> (?)
... and 28 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 d8f677e...a34da84. Read the comment docs.

int srcJDBCTypeInt = srcMeta.getColumnType(i + 1);
int destJDBCTypeInt = destMeta.getColumnType(i + 1);

// varify column types

Choose a reason for hiding this comment

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

typo: verify

@xiangyushawn xiangyushawn merged commit 5789e9a into microsoft:RTW_6.2.0 Jun 27, 2017
@xiangyushawn xiangyushawn deleted the TVP-ALL-Types-Test branch June 27, 2017 23:33
@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

5 participants