Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(456)

Unified Diff: service/datastore/raw_interface.go

Issue 2302743002: Interface update, per-method Contexts. (Closed)
Patch Set: WithoutTransaction, comments, fixes, cleanup. Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: service/datastore/raw_interface.go
diff --git a/service/datastore/raw_interface.go b/service/datastore/raw_interface.go
index 96e54d3370311ffb63b4a2e01065ab94d0a2bc80..dbfab19827220bbc7b0d6f9d80147265139e1594 100644
--- a/service/datastore/raw_interface.go
+++ b/service/datastore/raw_interface.go
@@ -184,7 +184,14 @@ type RawInterface interface {
// - cb is not nil
DeleteMulti(keys []*Key, cb DeleteMultiCB) error
- // Testable returns the Testable interface for the implementation, or nil if
- // there is none.
- Testable() Testable
+ // WithoutTransaction returns a derived Context without a transaction applied.
iannucci 2016/09/16 07:39:43 Note that it's safe to call this even when the con
dnj 2016/09/16 16:09:21 Done. I also made the user-facing implementation o
+ WithoutTransaction() context.Context
+
+ // CurrentTransaction returns a reference to the current Transaction, or nil
+ // if the Context does not have a current Transaction.
+ CurrentTransaction() Transaction
+
+ // GetTestable returns the Testable interface for the implementation, or nil
+ // if there is none.
+ GetTestable() Testable
}

Powered by Google App Engine
This is Rietveld 408576698