Skip to content
This repository was archived by the owner on Aug 29, 2025. It is now read-only.

Update from remote branch#5

Open
ahmb84 wants to merge 33 commits intoformalco:mainfrom
pganalyze:main
Open

Update from remote branch#5
ahmb84 wants to merge 33 commits intoformalco:mainfrom
pganalyze:main

Conversation

@ahmb84
Copy link

@ahmb84 ahmb84 commented Jan 31, 2025

No description provided.

msepga and others added 30 commits May 15, 2023 15:04
This fixes the warnings that arise in #81
…#84)

In passing, add an explanatory note on why the warnings are being
ignored, and that we can remove this once upgrading the parser to PG 16.
* Update `libpg_query` to `15-4.2.3`

* Update `CHANGELOG.md`
This was still referencing `?` replacement characters, which haven't been used for many years now.
In passing this updates the structure of the include files, as done in the libpg_query source.

Windows support requires Go 1.20+ due to golang/go#57455
These were accidentally copied over from the Ruby library, and don't
apply to Go.
* Bump libpg_query tag

* Update sources

* Remove invalid comment in testdata JSON

* Update expected parse test results

* Update module to v6

* Update benchmarks

* Update CHANGELOG
This adds extra `gokeep.go` files into each directory that only contains C source code, to ensure Go's "go mod vendor" command retains the source files correctly. Previously this required use of other vendor tools like modvendor, which are now no longer required.
This fixes strchrnul compatibility on macOS 15.4+
This fails the build and needs to be adjusted in libpg_query, but for now
just remove it manually.
Co-authored-by: Gui Lin <guileen@gmail.com>
…#146)

This avoids having to deal with Protobuf serialization/deserialization
when the caller is only interested to know if the input statements are
utility statements. As a side effect, this method also allows inferring
the number of statements (separated by ";") in the input.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants