cloud_functions: totals cache should be overwritten for skipDates (wo…

…rmhole-foundation#1147)
rh-low · May 3, 2022 · 3023b4a · 3023b4a
1 parent ec87530
commit 3023b4a
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/event_database/cloud_functions/totals.go b/event_database/cloud_functions/totals.go
@@ -153,14 +153,15 @@ func createCountsOfInterval(tbl *bigtable.Table, ctx context.Context, prefix str
 				results[dateStr][countBy] = results[dateStr][countBy] + 1
 			}
 
-			if cacheData, ok := warmTotalsCache[dateStr][cachePrefix]; !ok || len(cacheData) <= 1 {
-				// set the result in the cache
+			if daysAgo >= 1 {
 				muWarmTotalsCache.Lock()
-				warmTotalsCache[dateStr][cachePrefix] = results[dateStr]
+				if cacheData, ok := warmTotalsCache[dateStr][cachePrefix]; !ok || len(cacheData) <= 1 || !useCache(dateStr) {
+					// set the result in the cache
+					warmTotalsCache[dateStr][cachePrefix] = results[dateStr]
+					cacheNeedsUpdate = true
+				}
 				muWarmTotalsCache.Unlock()
-				cacheNeedsUpdate = true
 			}
-
 		}(tbl, ctx, prefix, daysAgo)
 	}