Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@olavloite
Copy link

A connection can execute several different types of transactions and batches (DML/DDL). These all implement the base interface UnitOfWork.

@olavloite olavloite requested a review from kolea2 July 25, 2019 16:32
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 25, 2019
import com.google.cloud.spanner.ReadContext.QueryAnalyzeMode;

/**
* {@link AnalyzeMode} indicates whether a query should be executed as a normal query, whether only
Copy link
Contributor

Choose a reason for hiding this comment

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

would add "as a normal query (NONE)..."

Copy link
Author

Choose a reason for hiding this comment

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

done

ParsedStatement statement, AnalyzeMode analyzeMode, QueryOption... options);

/**
* @return the read timestamp of this transaction. Will throw an {@link SpannerException} if there
Copy link
Contributor

Choose a reason for hiding this comment

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

minor - "Will throw a" instead of "an"

Copy link
Author

Choose a reason for hiding this comment

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

done

@codecov
Copy link

codecov bot commented Jul 25, 2019

Codecov Report

Merging #5881 into spanner-jdbc will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@                Coverage Diff                 @@
##             spanner-jdbc    #5881      +/-   ##
==================================================
+ Coverage           46.12%   46.12%   +<.01%     
- Complexity          24187    24189       +2     
==================================================
  Files                2456     2456              
  Lines              262236   262236              
  Branches            29602    29602              
==================================================
+ Hits               120947   120952       +5     
+ Misses             132176   132173       -3     
+ Partials             9113     9111       -2
Impacted Files Coverage Δ Complexity Δ
...age/contrib/nio/SeekableByteChannelPrefetcher.java 66.25% <0%> (+3.12%) 23% <0%> (+2%) ⬆️

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 2d7750f...0cf0ef7. Read the comment docs.

@olavloite olavloite requested a review from kolea2 July 26, 2019 04:55
@olavloite olavloite merged commit 3cec973 into googleapis:spanner-jdbc Jul 26, 2019
olavloite added a commit to olavloite/google-cloud-java that referenced this pull request Aug 5, 2019
…tions (googleapis#5881)

add base interface for unit of work on connections
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants