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

Unified Diff: filter/txnBuf/doc.go

Issue 2302743002: Interface update, per-method Contexts. (Closed)
Patch Set: Lightning talk licenses. 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
« no previous file with comments | « filter/txnBuf/context.go ('k') | filter/txnBuf/ds_txn.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: filter/txnBuf/doc.go
diff --git a/filter/txnBuf/doc.go b/filter/txnBuf/doc.go
index b20dba4a22f953ad46a327a58a0cc3781886cc54..66296376247238505204afd7ee5453755b55d8aa 100644
--- a/filter/txnBuf/doc.go
+++ b/filter/txnBuf/doc.go
@@ -73,4 +73,10 @@
// - The changing of namespace inside of a transaction is undefined... This is
// just generally a terrible idea anyway, but I thought it was worth
// mentioning.
+//
+// - Currently, the soft transactions are not directly accessible using the
+// CurrentTransaction interface; it returns the wrapped datastore's
+// transaction. While this is still correct, it could definitely be made
+// more useful by adding transaction buffer metadata to the returned
+// object.
package txnBuf
« no previous file with comments | « filter/txnBuf/context.go ('k') | filter/txnBuf/ds_txn.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698