Skip to content

spanner-jdbc: Step 14 - Add base interface for unit of work on connections - #5881

Merged
olavloite merged 3 commits into
googleapis:spanner-jdbcfrom
olavloite:spanner-jdbc-14-unit-of-work
Jul 26, 2019
Merged

spanner-jdbc: Step 14 - Add base interface for unit of work on connections#5881
olavloite merged 3 commits into
googleapis:spanner-jdbcfrom
olavloite:spanner-jdbc-14-unit-of-work

Conversation

@olavloite

Copy link
Copy Markdown
Contributor

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 kolea2July 25, 2019 16:32
@googlebotgooglebot 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
Copy Markdown
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
Copy Markdown
ContributorAuthor

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
Copy Markdown
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
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

done

@codecov

codecovBot commented Jul 25, 2019

Copy link
Copy Markdown

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 FilesCoverage ΔComplexity Δ
...age/contrib/nio/SeekableByteChannelPrefetcher.java66.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 kolea2July 26, 2019 04:55
@olavloite
olavloite merged commit 3cec973 into googleapis:spanner-jdbcJul 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 freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@olavloite@kolea2@googlebot