Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop daily_sorted_letter table #4314

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

klssmith
Copy link
Contributor

We stopped writing to this table when we started receiving letter delivery status via HTTPS callback instead of via response files in S3. However, since 2023 we've been using the ft_billing_letter_despatch table for letter billing and to store more detailed information about which letters were classed as sorted vs unsorted so this data is not needed.

@klssmith
Copy link
Contributor Author

Squawk warnings are fine:

note: Deleting an index blocks selects, inserts, updates, and deletes on the index's table.
  help: Delete the index CONCURRENTLY.

note: Dropping a table may break existing clients.

Nothing is using the table, so breaking existing clients or blocking writes to it while the index is dropped isn't a concern

@klssmith klssmith marked this pull request as ready for review December 18, 2024 15:02
@klssmith klssmith force-pushed the drop-daily-sorted-letter branch from 9ec7619 to d697fce Compare December 30, 2024 09:26
We stopped writing to this table when we started receiving letter
delivery status via HTTPS callback instead of via response files in S3.
However, since 2023 we've been using the `ft_billing_letter_despatch`
table for letter billing and to store more detailed information about
which letters were classed as `sorted` vs `unsorted` so this data is not
needed.
@klssmith klssmith force-pushed the drop-daily-sorted-letter branch from d697fce to 8a0db0a Compare January 2, 2025 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant