From cd22818240455cde8acc3010ce03f724bc3e2c7e Mon Sep 17 00:00:00 2001 From: Eli Ribble Date: Wed, 14 Jan 2026 21:52:36 +0000 Subject: [PATCH] Actually exit early on empty aggregate Previous attempt was subtly wrong. --- background/arcgis.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/background/arcgis.go b/background/arcgis.go index 6997a2ec..6a32ec53 100644 --- a/background/arcgis.go +++ b/background/arcgis.go @@ -1039,6 +1039,10 @@ func updateSummaryTables(ctx context.Context, org *models.Organization) { log.Error().Err(err).Msg("Failed to get organization") return } + if len(point_locations) == 0 { + log.Info().Int("org_id", int(org.ID)).Msg("No updates to perform") + return + } log.Info().Int("count", len(point_locations)).Msg("Summarizing point locations") for i := range 16 { @@ -1075,10 +1079,6 @@ func updateSummaryTables(ctx context.Context, org *models.Organization) { im.SetCol("count_").To(psql.Raw("EXCLUDED.count_")), )) //log.Info().Str("sql", insertQueryToString(psql.Insert(to_insert...))).Msg("Updating...") - if len(to_insert) == 0 { - log.Info().Int("resolution", i).Msg("No updates to perform") - continue - } _, err := psql.Insert(to_insert...).Exec(ctx, db.PGInstance.BobDB) if err != nil { log.Error().Err(err).Msg("Faild to add h3 aggregation")