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 operators used by ltree extension #136

Merged
merged 1 commit into from
Apr 29, 2022

Conversation

svenklemm
Copy link
Contributor

This makes libpg_query no longer error on the following operators
used by the ltree extension: ?<@ ?@> ?~ ?@ ^?

This makes libpg_query no longer error on the following operators
used by the ltree extension: ?<@ ?@> ?~ ?@ ^?
Copy link
Member

@lfittl lfittl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lfittl lfittl merged commit 5ddcf81 into pganalyze:13-latest Apr 29, 2022
@ghost ghost mentioned this pull request Jun 14, 2022
lfittl added a commit to pganalyze/collector that referenced this pull request Jun 28, 2022
Relevant changes that apply to the collector:

- Fixes memory leak in pg_query_fingerprint error handling [#141](pganalyze/libpg_query#141)
- Add support for parsing more operators that include a `?` character (special cased to support old pg_stat_statements query texts)
  - ltree extension [#136](pganalyze/libpg_query#136)
  - promscale extension [#133](pganalyze/libpg_query#133)
lfittl added a commit to pganalyze/collector that referenced this pull request Jun 29, 2022
Relevant changes that apply to the collector:

- Fixes memory leak in pg_query_fingerprint error handling [#141](pganalyze/libpg_query#141)
- Add support for parsing more operators that include a `?` character (special cased to support old pg_stat_statements query texts)
  - ltree extension [#136](pganalyze/libpg_query#136)
  - promscale extension [#133](pganalyze/libpg_query#133)
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

Successfully merging this pull request may close these issues.

2 participants