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

fix: Use shortestName instead of sequenceName for Postgres seqence migration #36791

Closed
wants to merge 2 commits into from

Conversation

jbmolle
Copy link

@jbmolle jbmolle commented Feb 21, 2023

Summary

Sequence migration fails on Postgres 15 because the sequenceName contains Postgres schema name which is not public.
Access to the public schema has been removed in Postgres 15. If a schema with the name of the user exists, it is going to be used per default. The sequenceName contains the schema name and it is better to use the shortestName.

Checklist

@szaimen szaimen added the 3. to review Waiting for reviews label Feb 21, 2023
@szaimen szaimen added this to the Nextcloud 27 milestone Feb 21, 2023
@szaimen szaimen requested review from a team, ArtificialOwl, icewind1991 and blizzz and removed request for a team February 21, 2023 08:37
…name is not

referenced in column_default

Signed-off-by: Jean-Baptiste Molle <jb.molle@vescape.com>
Signed-off-by: Jean-Baptiste Molle <jb.molle@vescape.com>
@skjnldsv skjnldsv mentioned this pull request May 3, 2023
This was referenced May 9, 2023
@blizzz blizzz mentioned this pull request May 17, 2023
@blizzz blizzz modified the milestones: Nextcloud 27, Nextcloud 28 May 23, 2023
@skjnldsv skjnldsv mentioned this pull request Nov 1, 2023
This was referenced Nov 6, 2023
This was referenced Nov 14, 2023
@blizzz blizzz modified the milestones: Nextcloud 28, Nextcloud 29 Nov 23, 2023
This was referenced Mar 12, 2024
This was referenced Mar 20, 2024
@skjnldsv skjnldsv mentioned this pull request Mar 28, 2024
81 tasks
@skjnldsv skjnldsv modified the milestones: Nextcloud 29, Nextcloud 30 Mar 28, 2024
@skjnldsv skjnldsv added 2. developing Work in progress stale Ticket or PR with no recent activity and removed 3. to review Waiting for reviews labels Jul 27, 2024
This was referenced Jul 30, 2024
This was referenced Aug 5, 2024
@skjnldsv skjnldsv mentioned this pull request Aug 13, 2024
@skjnldsv skjnldsv closed this Aug 14, 2024
@skjnldsv skjnldsv removed this from the Nextcloud 30 milestone Aug 14, 2024
@skjnldsv
Copy link
Member

Hey @jbmolle! We're actively testing our server with pgsql from 12 to 16 (at the time of this comment), is there still an issue with this then?
I'm not too much of an expert in Postgres, but it looks like we have no issues with Postgres 15 and 16 🤔

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress feature: occ stale Ticket or PR with no recent activity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Migration to Postgresql 15 fails
6 participants