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
Add workflow to test PG14 #3349
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3349 +/- ##
==========================================
- Coverage 90.61% 90.59% -0.03%
==========================================
Files 212 212
Lines 35588 35592 +4
==========================================
- Hits 32248 32243 -5
- Misses 3340 3349 +9
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few minor comments, but I mostly wonder if we should run it on a schedule. It could cause a lot of extra noise because of flakes and I do not think it is needed.
@@ -96,6 +96,7 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS ON) | |||
|
|||
# Code coverage is optional and OFF by default | |||
option(CODECOVERAGE "Enable code coverage for the build" OFF) | |||
option(EXPERIMENTAL "Skip postgres version compatibility check" OFF) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
... or DEVELOPMENT
. No opinion, just offering alternatives.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
e30553d
to
e5db993
Compare
This patch adds a new EXPERIMENTAL flag to cmake allowing skipping the check for a compatible postgres version. It also adds macros needed for PG14 support.
PG14 renames field relkind for CreateTableAsStmt and AlterTableStmt to objtype. postgres/postgres@cc35d8933a
PG14 changes xact.h to no longer include fmgr.h which is needed for PG_FUNCTION_ARGS definition. This patch includes fmgr.h explicitly to no longer rely on the indirect include. postgres/postgres@3941eb6341
Disable-check: commit-count