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

Upgrade to more recent postgres version #1483

Closed
uweschaefer opened this issue Aug 4, 2021 · 6 comments · Fixed by #2303
Closed

Upgrade to more recent postgres version #1483

uweschaefer opened this issue Aug 4, 2021 · 6 comments · Fixed by #2303
Assignees

Comments

@uweschaefer
Copy link
Member

currently using 11.5. We'd need to make sure that more recent versions work as expected.

uweschaefer added a commit that referenced this issue Aug 4, 2021
@uweschaefer uweschaefer removed the notNow label Jun 6, 2022
@bedaka
Copy link
Contributor

bedaka commented Apr 28, 2023

Will try to move 15.2 (latest supported by RDS)

@bedaka bedaka self-assigned this Apr 28, 2023
@uweschaefer
Copy link
Member Author

uweschaefer commented Apr 28, 2023

There is a snag to this: (Credits to @mweirauch )

2023-02-22 09:42:58.511 ERROR 101 — [ main]
liquibase.changelog.ChangeSet : Change Set
classpath:/db/changelog/db.changelog-master.yaml::transformationcache_and_schemastore_col_to_jsonb::uweschaefer
failed. Error: ERROR: DISCARD ALL cannot be executed within a pipeline
[Failed SQL: (0) ALTER TABLE fact ALTER COLUMN header SET STORAGE
MAIN, ALTER COLUMN payload SET STORAGE EXTENDED;

A similar report:
liquibase/liquibase#3806
related (and linked from above issue):
DbUp/dbup-postgresql#10
https://www.postgresql.org/message-id/4139340.1676050723%40sss.pgh.pa.us

@uweschaefer
Copy link
Member Author

liquibase seems to use a batch, where it should not.

@mweirauch
Copy link
Member

Any recently released (2023-03) minor of PostgreSQL is affected. We trapped into this with 13.10. Unfortunately that liquibase issue didn't get much traction, yet.

(Sorry for not creating the ticket when you asked for Uwe. Life's kinda busy over here.)

@bedaka bedaka removed their assignment May 2, 2023
@bedaka
Copy link
Contributor

bedaka commented May 2, 2023

In this case we'll pause this issue to see if they can come up with a solution.

otbe added a commit that referenced this issue Jun 14, 2023
otbe added a commit that referenced this issue Jun 14, 2023
otbe added a commit that referenced this issue Jun 14, 2023
@otbe
Copy link
Member

otbe commented Jun 14, 2023

I started working on a workaround for the liquibase issue and some other minor fixes for PG15 compatibility. Stay tuned 🥳

@otbe otbe self-assigned this Jun 14, 2023
otbe added a commit that referenced this issue Jun 15, 2023
Merge remote-tracking branch 'origin/master' into issue1483-fix-liquibase

# Conflicts:
#	factcast-itests/factcast-itests-factus/src/test/java/org/factcast/itests/factus/client/GrpcThreadingITest.java
#	factcast-itests/factcast-itests-store/src/main/java/org/factcast/itests/store/IntegrationTestContext.java
#	factcast-itests/factcast-itests-tls/src/test/java/org/factcast/itests/tls/test/TLSClientTest.java
#	factcast-store/src/test/java/org/factcast/store/internal/PgTestConfiguration.java
#	factcast-test/src/main/java/org/factcast/test/FactcastTestConfig.java
otbe added a commit that referenced this issue Jun 15, 2023
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 a pull request may close this issue.

5 participants