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

PG 14 upgrade #92

Open
pyramation opened this issue Dec 22, 2022 · 5 comments
Open

PG 14 upgrade #92

pyramation opened this issue Dec 22, 2022 · 5 comments

Comments

@pyramation
Copy link
Collaborator

pyramation commented Dec 22, 2022

[email protected]

@eljefedelrodeodeljefe
Copy link

@pyramation do you have plans on integrating it? If you need help what would be a one-liner statement of what needs to be done?

@pyramation
Copy link
Collaborator Author

When I attempted this, the string type was changed — so some of the messages and node parsing needed to be updated. Would love to take a PR for this — and it would also allow for 15 to happen as well.

@nith2001
Copy link

@pyramation Hello! Not sure what you were referring to by the string type, but does it related to the libpg-query-node repository needing to upgrade to V15? Relevant PR here.

@pyramation
Copy link
Collaborator Author

@pyramation Hello! Not sure what you were referring to by the string type, but does it related to the libpg-query-node repository needing to upgrade to V15? Relevant PR here.

yep, the output from libpq-query-node produces a different JSON structure than before for some of the primitive node types — which I believe was the raw string node. This one is particularly everywhere in the pgsql-parser deparser code. One idea for a first attempt is to preparse the output JSON so it conforms to the current codebase and current deparser, and later we can update the deparser.

@nith2001
Copy link

nith2001 commented Apr 19, 2023

Ahh, I see. So the idea would be to write some code in libpq-query-node to take the output from the original libpq-query code in Version 15, pre-parse it, and return that to pgsql-parser whenever it is called?

If so, that would definitely work, but would complications arise in other codebases where the libpq-query-node library is being used? Or does that not matter because the only way that would happen is for other codebases to update their version?

Edit: Also ngl, I've almost never worked in C++ so I feel like my learning curve would be a tad steep to try and help contribute to libpg-query-node. If there's anything I can do to help on the Javascript side of things on pgsql-parser like writing tests, I'd be down to help.

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

No branches or pull requests

3 participants