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

Add appendinfo.h and regenerate bindings #479

Merged
merged 2 commits into from
Mar 15, 2022

Conversation

einarmo
Copy link
Contributor

@einarmo einarmo commented Mar 14, 2022

Seems to have upgraded bindings to 12.10 from 12.9, among other things.

appendinfo.h contains a couple of methods used for foreign data wrappers. In particular add_row_identity_var is referenced from the official postgres documentation, and is nice to have here.

The file exists for pg12 and pg13 as well, but it's just the add_row_identity_var method first added in pg14 which seem generally useful here. I think only that method is needed for external applications in practice.

Seems to have upgraded bindings to 12.10 from 12.9.

appendinfo.h contains a couple of methods used for foreign data
wrappers. In particular `add_row_identity_var` is referenced from the
official postgres documentation, and is nice to have here.
@Hoverbear
Copy link
Contributor

@einarmo this looks good overall, however it is kind of our policy to include the same things wherever possible.

Could I ask you to add the includes for pg12 and pg13 as well?

It does not seem to exist for pg10 and pg11.
@Hoverbear Hoverbear merged commit daa0ecf into pgcentralfoundation:develop Mar 15, 2022
@Hoverbear
Copy link
Contributor

Thanks!

@einarmo einarmo deleted the add-appendinfo-h branch March 15, 2022 19:10
@Hoverbear Hoverbear mentioned this pull request Mar 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants