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

feat: add support for PostgreSQL 16 #208

Closed
wants to merge 1 commit into from

Conversation

gbartolini
Copy link

As part of the CloudNativePG community we are building Postgres containers from the current trunk of the source code (what will be PostgreSQL 16). This is the project: https://github.com/cloudnative-pg/postgres-trunk-containers

In December the pipeline started to show errors due to failures with the pgaudit extension compilation, after the "Rework query relation permission checking" patch was committed.

This patch is my attempt to fix the issue, but I could only solve the compilation issue. When I test it with PostgreSQL 16, I run into an issue with memory deallocation that causes the server to crash at start time.

Unfortunately, my skills and time availability do now allow me to go any further. But at least I hope that this could serve as a starting point for future integration. Thanks anyway.

Signed-off-by: Gabriele Bartolini <gabriele.bartolini@enterprisedb.com>
@dwsteele dwsteele self-requested a review March 9, 2023 16:35
@dwsteele dwsteele self-assigned this Mar 9, 2023
@gbartolini gbartolini mentioned this pull request Jun 7, 2023
@dwsteele dwsteele mentioned this pull request Jun 9, 2023
@dwsteele
Copy link
Contributor

dwsteele commented Jun 9, 2023

Closing this in favor of #216 which contains all changes required for PG16. Thank you for getting this started, @gbartolini!

@dwsteele dwsteele closed this Jun 9, 2023
dwsteele added a commit that referenced this pull request Aug 3, 2023
Permissions were split out from RangeTblEntry in https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=a61b1f74823c so add that lookup for any RTEs that pass the initial filters.

Update GUC memory management as required by https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=407b50f2d421bca5.

Remove bms_first_member() as required by https://git.postgresql.org/gitweb/?p=postgresql.git;a=commit;h=462bb7f12851c.

NOTE: Some preliminary work on permissions was contributed by @gbartolini in #208.
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.

None yet

2 participants