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

Unified Diff: impl/prod/memcache.go

Issue 1490243004: Make appengine connection explicitly owned by impl/prod. (Closed) Base URL: https://github.com/luci/gae.git@master
Patch Set: fix other services too Created 5 years 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: impl/prod/memcache.go
diff --git a/impl/prod/memcache.go b/impl/prod/memcache.go
index 90560b2f0b2ee6f5d9608cde68a85e63606c0637..571eec8beb31bf95361c0985f6b4f4a1392f164b 100644
--- a/impl/prod/memcache.go
+++ b/impl/prod/memcache.go
@@ -17,11 +17,13 @@ import (
// by gae.GetMC(c)
func useMC(c context.Context) context.Context {
return mc.SetRawFactory(c, func(ci context.Context) mc.RawInterface {
- return mcImpl{ci}
+ return mcImpl{AEContext(ci)}
})
}
-type mcImpl struct{ context.Context }
+type mcImpl struct {
+ aeCtx context.Context
+}
type mcItem struct {
i *memcache.Item
@@ -107,19 +109,19 @@ func doCB(err error, cb mc.RawCB) error {
}
func (m mcImpl) DeleteMulti(keys []string, cb mc.RawCB) error {
- return doCB(memcache.DeleteMulti(m.Context, keys), cb)
+ return doCB(memcache.DeleteMulti(m.aeCtx, keys), cb)
}
func (m mcImpl) AddMulti(items []mc.Item, cb mc.RawCB) error {
- return doCB(memcache.AddMulti(m.Context, mcMF2R(items)), cb)
+ return doCB(memcache.AddMulti(m.aeCtx, mcMF2R(items)), cb)
}
func (m mcImpl) SetMulti(items []mc.Item, cb mc.RawCB) error {
- return doCB(memcache.SetMulti(m.Context, mcMF2R(items)), cb)
+ return doCB(memcache.SetMulti(m.aeCtx, mcMF2R(items)), cb)
}
func (m mcImpl) GetMulti(keys []string, cb mc.RawItemCB) error {
- realItems, err := memcache.GetMulti(m.Context, keys)
+ realItems, err := memcache.GetMulti(m.aeCtx, keys)
if err != nil {
return err
}
@@ -135,22 +137,22 @@ func (m mcImpl) GetMulti(keys []string, cb mc.RawItemCB) error {
}
func (m mcImpl) CompareAndSwapMulti(items []mc.Item, cb mc.RawCB) error {
- return doCB(memcache.CompareAndSwapMulti(m.Context, mcMF2R(items)), cb)
+ return doCB(memcache.CompareAndSwapMulti(m.aeCtx, mcMF2R(items)), cb)
}
func (m mcImpl) Increment(key string, delta int64, initialValue *uint64) (uint64, error) {
if initialValue == nil {
- return memcache.IncrementExisting(m.Context, key, delta)
+ return memcache.IncrementExisting(m.aeCtx, key, delta)
}
- return memcache.Increment(m.Context, key, delta, *initialValue)
+ return memcache.Increment(m.aeCtx, key, delta, *initialValue)
}
func (m mcImpl) Flush() error {
- return memcache.Flush(m.Context)
+ return memcache.Flush(m.aeCtx)
}
func (m mcImpl) Stats() (*mc.Statistics, error) {
- stats, err := memcache.Stats(m.Context)
+ stats, err := memcache.Stats(m.aeCtx)
if err != nil {
return nil, err
}
« impl/prod/datastore_key.go ('K') | « impl/prod/info.go ('k') | impl/prod/raw_datastore.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698