Fix concurrent map write (#292)

* Fix concurrent map write
interline-io · Dec 6, 2024 · ccbb33c · ccbb33c
1 parent 7b5d4f0
commit ccbb33c
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
diff --git a/cmd/tlserver/main.go b/cmd/tlserver/main.go
@@ -106,7 +106,6 @@ func (cmd *ServerCommand) AddFlags(fl *pflag.FlagSet) {
 	fl.BoolVar(&cmd.LoadAdmins, "load-admins", false, "Load admin polygons from database into memory")
 	fl.IntVar(&cmd.LoaderBatchSize, "loader-batch-size", 100, "GraphQL Loader batch size")
 	fl.IntVar(&cmd.LoaderStopTimeBatchSize, "loader-stop-time-batch-size", 1, "GraphQL Loader batch size for StopTimes")
-
 }
 
 func (cmd *ServerCommand) Parse(args []string) error {

diff --git a/internal/clock/swcache.go b/internal/clock/swcache.go
@@ -36,8 +36,8 @@ func NewServiceWindowCache(db sqlx.Ext) *ServiceWindowCache {
 
 func (f *ServiceWindowCache) Get(ctx context.Context, fvid int) (ServiceWindow, bool, error) {
 	f.lock.Lock()
+	defer f.lock.Unlock()
 	a, ok := f.fvslWindows[fvid]
-	f.lock.Unlock()
 	if ok {
 		return a, ok, nil
 	}