fix(aci): Delete rows from GroupOpenPeriod

[snigdhas]

Need to iterate over the rows because there's too many to simply .all().delete()

[github-actions]

This PR has a migration; here is the generated SQL for src/sentry/migrations/0915_delete_open_period_rows.py

for 0915_delete_open_period_rows in sentry

--
-- Raw Python operation
--
-- THIS OPERATION CANNOT BE WRITTEN AS SQL

[vgrozdanic]

The code looks good, but can you please provide some more context around this, why is this being done?

I see a PR opened before this one to backfill the data into this table.

Are we going to delete all of the data, and then do the backfilling?

[saponifi3d]

Are we going to delete all of the data, and then do the backfilling?

@vgrozdanic - yeah that's the plan, the migration had some missing data so trying again.

---

PR lgtm from the product side.

[github-actions]

This PR has a migration; here is the generated SQL for src/sentry/migrations/0916_delete_open_period_rows.py

for 0916_delete_open_period_rows in sentry

--
-- Raw Python operation
--
-- THIS OPERATION CANNOT BE WRITTEN AS SQL

[snigdhas]

@vgrozdanic we had a bug in the logic to write to this table hence the need to delete the rows. The data in GroupOpenPeriod isn't used anywhere yet, so the plan is to delete the rows, resume writes (feature-flagged), then backfill for existing groups

[vgrozdanic]

@vgrozdanic we had a bug in the logic to write to this table hence the need to delete the rows. The data in GroupOpenPeriod isn't used anywhere yet, so the plan is to delete the rows, resume writes (feature-flagged), then backfill for existing groups

Got it, thanks for clarifying! 🙏