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

DBZ-8275 Support for PostgreSQL 17 #400

Merged
merged 2 commits into from
Oct 16, 2024

Conversation

dargmuesli
Copy link
Contributor

@dargmuesli dargmuesli commented Oct 3, 2024

https://issues.redhat.com/browse/DBZ-8275

Depends on debezium/postgres-decoderbufs#31

Taking inspiration from c32a5cf, I've added files for postgres v17. It might be possible to use Debian bookworm instead of bullseye as base image, but that's maybe for another discussion.

@Naros
Copy link
Member

Naros commented Oct 4, 2024

Hi @dargmuesli, can you please file a Jira issue and prepend the Jira issue key to the commit message? Thanks!

@jpechane
Copy link
Contributor

jpechane commented Oct 4, 2024

There is https://issues.redhat.com/browse/DBZ-8275 but additional changes are needed. See description

@dargmuesli dargmuesli changed the title feat(postgres): add v17 DBZ-8275 add postgres v17 Oct 4, 2024
@dargmuesli dargmuesli changed the title DBZ-8275 add postgres v17 DBZ-8275 Add postgres v17 Oct 4, 2024
@dargmuesli dargmuesli changed the title DBZ-8275 Add postgres v17 DBZ-8275 Support for PostgreSQL 17 Oct 4, 2024
@dargmuesli
Copy link
Contributor Author

Thank you for linking, I updated the PR and commit title. Not sure where to make the other changes, likely another repo, I suppose. Also I'm not into decoderbufs, so I cannot help there, I think. Please take this PR as a first step into PostgreSQL v17 support and feel free to build upon it :)

@jpechane jpechane merged commit db66c97 into debezium:main Oct 16, 2024
4 of 5 checks passed
@dargmuesli dargmuesli deleted the feat/postgres/v17 branch October 16, 2024 08:59
@dargmuesli
Copy link
Contributor Author

@jpechane ah, should build-all.sh haven been updates as well maybe? As I don't see 17-alpine on docker hub right now, just 17.

@dargmuesli
Copy link
Contributor Author

Oh, i just saw

So I switched to quay.io 👍

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.

3 participants