Skip to content

Commit

Permalink
chore(program_v2): log stale dimension value cleanup better
Browse files Browse the repository at this point in the history
  • Loading branch information
japsu committed Jun 15, 2024
1 parent fad3f3f commit ac42de2
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions backend/program_v2/models/dimension.py
Original file line number Diff line number Diff line change
Expand Up @@ -284,10 +284,8 @@ def save_many(

for dim_dto, dim_dj in zip(dimension_dtos, django_dimensions, strict=True):
values_to_keep = [choice.slug for choice in dim_dto.choices or []]
num_deleted_dvs, _ = (
DimensionValue.objects.filter(dimension=dim_dj).exclude(slug__in=values_to_keep).delete()
)
logger.info("Deleted %s stale dimension values for dimension %s", num_deleted_dvs, dim_dj)
_, deleted = dim_dj.values.exclude(slug__in=values_to_keep).delete()
logger.info("Stale dimension value cleanup for %s deleted %s", dim_dto.slug, deleted or "nothing")

if remove_others:
dimensions_to_keep = [dim_dto.slug for dim_dto in dimension_dtos]
Expand Down

0 comments on commit ac42de2

Please sign in to comment.