Skip to content

Bigint Migration for 'events' Table (Step 3) #4406

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

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

Conversation

philippthun
Copy link
Member

@philippthun philippthun commented Jun 13, 2025

  • implementation of steps 3a and 3b for the events table
  • step 3a: add check constraint
  • step 3b: drop check constraint and trigger function / drop old id column / switch id_bigint -> id
  • down migration fully implemented for testing purposes
  • added comments to step 1 and step 3b down migrations - they should not be used in a production system
  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@philippthun philippthun force-pushed the bigint-migration-events-step3 branch 2 times, most recently from 6aaf1df to 29d1456 Compare June 26, 2025 12:51
@philippthun philippthun force-pushed the bigint-migration-events-step3 branch from 29d1456 to 5ef5cc8 Compare July 3, 2025 09:24
@philippthun philippthun force-pushed the bigint-migration-events-step3 branch from 5ef5cc8 to e8e1df1 Compare July 10, 2025 11:12
- implementation of steps 3a and 3b for the events table
- step 3a: add check constraint
- step 3b: drop check constraint and trigger function / drop old id
  column / switch id_bigint -> id
- down migration fully implemented for testing purposes
- added comments to step 1 and step 3b down migrations - they should not
  be used in a production system
@philippthun philippthun force-pushed the bigint-migration-events-step3 branch from e8e1df1 to 4c39d2d Compare July 11, 2025 07:12
@philippthun philippthun marked this pull request as ready for review July 11, 2025 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant