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

Initialize TM for XAResource #257

Merged
merged 1 commit into from
Apr 24, 2017

Conversation

Suraiya-Hameed
Copy link
Contributor

@Suraiya-Hameed Suraiya-Hameed commented Apr 18, 2017

For every XAResource, initialize the TM by calling xa_open_entry

@codecov-io
Copy link

codecov-io commented Apr 18, 2017

Codecov Report

Merging #257 into dev will increase coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev     #257      +/-   ##
============================================
+ Coverage     33.77%   33.78%   +<.01%     
- Complexity     1522     1523       +1     
============================================
  Files           101      101              
  Lines         23600    23599       -1     
  Branches       3876     3875       -1     
============================================
+ Hits           7972     7974       +2     
+ Misses        14061    14056       -5     
- Partials       1567     1569       +2
Flag Coverage Δ Complexity Δ
#JDBC41 33.61% <0%> (-0.04%) 1517 <0> (+6)
#JDBC42 33.64% <0%> (+0.04%) 1513 <0> (-6) ⬇️
Impacted Files Coverage Δ Complexity Δ
.../microsoft/sqlserver/jdbc/SQLServerXAResource.java 0% <0%> (ø) 0 <0> (ø) ⬇️
src/main/java/microsoft/sql/DateTimeOffset.java 37.14% <0%> (-2.86%) 8% <0%> (-2%)
...om/microsoft/sqlserver/jdbc/ReaderInputStream.java 44.94% <0%> (-1.13%) 16% <0%> (-1%)
...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java 27.67% <0%> (-0.45%) 48% <0%> (-1%)
...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java 37.77% <0%> (-0.18%) 0% <0%> (ø)
...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java 34.71% <0%> (-0.07%) 0% <0%> (ø)
...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java 45.05% <0%> (+0.07%) 183% <0%> (ø) ⬇️
...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java 25.59% <0%> (+0.5%) 183% <0%> (+5%) ⬆️

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 c1f88c5...f276623. Read the comment docs.

@Suraiya-Hameed Suraiya-Hameed changed the title Initialize XA resource Initialize TM for XAResource Apr 18, 2017
@Suraiya-Hameed Suraiya-Hameed requested a review from v-nisidh April 19, 2017 16:34
@Suraiya-Hameed Suraiya-Hameed merged commit 77be8d7 into microsoft:dev Apr 24, 2017
@Suraiya-Hameed Suraiya-Hameed deleted the InitializeXA branch April 24, 2017 22:43
@v-nisidh v-nisidh added this to the 6.1.7 milestone Apr 28, 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.

6 participants