-
Notifications
You must be signed in to change notification settings - Fork 1.1k
spanner-jdbc: Step 14 - Add base interface for unit of work on connections #5881
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
spanner-jdbc: Step 14 - Add base interface for unit of work on connections #5881
Conversation
| import com.google.cloud.spanner.ReadContext.QueryAnalyzeMode; | ||
|
|
||
| /** | ||
| * {@link AnalyzeMode} indicates whether a query should be executed as a normal query, whether only |
There was a problem hiding this comment.
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)..."
There was a problem hiding this comment.
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 |
There was a problem hiding this comment.
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"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
…tions (googleapis#5881) add base interface for unit of work on connections
A connection can execute several different types of transactions and batches (DML/DDL). These all implement the base interface
UnitOfWork.