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

Side by Side Diff: filter/txnBuf/ds.go

Issue 1309803004: Add transaction buffer filter. (Closed) Base URL: https://github.com/luci/gae.git@add_query_support
Patch Set: Fix builtin+ancestor+multi-eq+multiIterator bug, add more txnBuf test Created 5 years, 2 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 unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 package txnBuf
6
7 import (
8 "golang.org/x/net/context"
9
10 ds "github.com/luci/gae/service/datastore"
11 )
12
13 // 10MB is the current initial limit
14 const transactionSizeLimit = 10 * 1000 * 1000
Vadim Sh. 2015/09/28 18:52:56 *1024*1024? :)
iannucci 2015/09/29 03:21:37 I was playing it safe. Not sure if they mean 1024
15
16 type dsBuf struct {
17 ds.RawInterface
18
19 ns string
20 }
21
22 var _ ds.RawInterface = (*dsBuf)(nil)
23
24 func (d *dsBuf) RunInTransaction(f func(context.Context) error, opts *ds.Transac tionOptions) error {
25 return doRunInTransaction(d.RawInterface, f, opts)
26 }
27
28 func doRunInTransaction(base ds.RawInterface, f func(context.Context) error, opt s *ds.TransactionOptions) error {
29 return base.RunInTransaction(func(ctx context.Context) error {
30 return withTxnBuf(ctx, f, opts)
31 }, opts)
32 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698