Skip to content

Transaction Documentation #623

Description

@stephenplusplus

Related: #621

Should we privatize transaction#begin and transaction#commit, removing them from the public docs and their usage in other Transaction examples?

A user goes through their Dataset instance to get a Transaction:

dataset.runInTransaction(transaction, done) {});

This calls transaction.begin immediately, and the done function returned to their callback is actually transaction.commit.

Having these methods appear in the docs is probably quite confusing, as the user shouldn't need (or be able to) begin a transaction that already began, or commit the transaction before calling done (commit) again.

Activity

  1. abelino commented on May 22, 2015

    @abelino

    Ohhh snap, this clears things up a lot!!! Thanks!!!

  2. ryanseys commented on May 22, 2015

    @ryanseys
    Contributor

    Yeah these should be private.

  3. added this to the Datastore Future milestone on May 25, 2015
  4. ryanseys commented on May 26, 2015

    @ryanseys
    Contributor

    Oooh I'll address these in #627

  5. added a commit that references this issue on Jan 28, 2026
  6. added a commit that references this issue on Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

api: datastoreIssues related to the Datastore API.

Type

No type

Projects

No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions