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

Unified Diff: luci_config/appengine/gaeconfig/default.go

Issue 2645763002: datastorecache: Add Locker field, public settings. (Closed)
Patch Set: Created 3 years, 11 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 | « luci_config/appengine/backend/datastore/ds.go ('k') | luci_config/appengine/gaeconfig/default_test.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: luci_config/appengine/gaeconfig/default.go
diff --git a/luci_config/appengine/gaeconfig/default.go b/luci_config/appengine/gaeconfig/default.go
index 859625392815ab35b172bcea5cf6d4762ef5310b..c12458f9aa2eeabb7285eebd87ffa7042196ae39 100644
--- a/luci_config/appengine/gaeconfig/default.go
+++ b/luci_config/appengine/gaeconfig/default.go
@@ -11,6 +11,7 @@ import (
"path/filepath"
"time"
+ "github.com/luci/luci-go/appengine/datastorecache"
"github.com/luci/luci-go/common/config/impl/filesystem"
"github.com/luci/luci-go/luci_config/appengine/backend/datastore"
"github.com/luci/luci-go/luci_config/appengine/backend/memcache"
@@ -121,10 +122,11 @@ func installConfigBackend(c context.Context, s *Settings, be backend.B, dsCron b
// If our datastore cache is enabled, install a handler for refresh. This
// will be loaded by dsCache's "HandlerFunc".
- if s.DatastoreCacheMode != dsCacheDisabled {
+ if s.DatastoreCacheMode != DSCacheDisabled {
dsc := datastore.Config{
RefreshInterval: exp,
- FailOpen: s.DatastoreCacheMode == dsCacheEnabled,
+ FailOpen: s.DatastoreCacheMode == DSCacheEnabled,
+ LockerFunc: datastorecache.MemLocker,
}
if !dsCron {
« no previous file with comments | « luci_config/appengine/backend/datastore/ds.go ('k') | luci_config/appengine/gaeconfig/default_test.go » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698