Skip to content

Recreate event trigger first in init.sql #948

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

Merged
merged 1 commit into from
Jul 11, 2025
Merged

Conversation

andrew-farries
Copy link
Collaborator

Move event trigger creation to be the first thing that happens (after pgroll schema creation) in init.sql.

Previous versions of pgroll used an event trigger that was incompatible with the latest version, causing errors on the upgrade path to pgroll 0.14. By moving event trigger creation to happen first during init, we ensure that the old event trigger installed by an older version of pgroll can't fire during the init of the new version.

@github-actions github-actions bot temporarily deployed to Docs Preview July 11, 2025 08:11 Inactive
@github-actions github-actions bot temporarily deployed to Docs Preview July 11, 2025 08:13 Inactive
@andrew-farries andrew-farries changed the base branch from main to disable-pgformatter July 11, 2025 08:25
@github-actions github-actions bot temporarily deployed to Docs Preview July 11, 2025 08:26 Inactive
@andrew-farries andrew-farries marked this pull request as ready for review July 11, 2025 08:36
@andrew-farries andrew-farries requested a review from kvch July 11, 2025 08:36
Base automatically changed from disable-pgformatter to main July 11, 2025 11:44
Move event trigger creation to be the first thing that happens (after
`pgroll` schema creation) in `init.sql`.

Previous versions of `pgroll` used an event trigger that was
incompatible with the latest version, causing errors on the upgrade path
to `pgroll 0.14`. By moving event trigger creation to happen first
during `init`, we ensure that the old event trigger installed by an
older version of `pgroll` can't fire during the `init` of the new
version.
@github-actions github-actions bot temporarily deployed to Docs Preview July 11, 2025 11:44 Inactive
@andrew-farries andrew-farries merged commit a906776 into main Jul 11, 2025
30 checks passed
@andrew-farries andrew-farries deleted the fix-upgrade-path branch July 11, 2025 11:48
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.

2 participants