Skip to content

common/spanner: introduce debug module and nested transaction warning #2517

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.

Sign up for GitHub

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

Merged
merged 5 commits into from
Oct 10, 2017
Merged

common/spanner: introduce debug module and nested transaction warning #2517

Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
8063199
common: expose debug module
stephenplusplus Aug 9, 2017
1663b09
spanner: use debug to communicate nested transaction warning
stephenplusplus Aug 9, 2017
f526847
just put it in the docs
stephenplusplus Sep 21, 2017
b900f6c
rm .only
stephenplusplus Sep 21, 2017
5fb7905
Update database.js
stephenplusplus Oct 10, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions packages/spanner/src/database.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -787,6 +787,8 @@ Database.prototype.runStream = function(query, options) {
* atomically at a single logical point in time across columns, rows, and tables
* in a database.
*
* Note that Cloud Spanner does not support nested transactions.

This comment was marked as spam.

Sign in to view

*
* The callback you provide to this function will become the "run function". It
* will be executed with either an error or a {module:spanner/transaction}
* object. The Transaction object will let you run queries and queue mutations
Expand Down