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

Don't bail on #? operator name #133

Merged
merged 1 commit into from
Apr 20, 2022
Merged

Conversation

svenklemm
Copy link
Contributor

The following statement will produce an error with pg_query_parse
while being accepted by PG13.

CREATE OPERATOR #? (LEFTARG=bool,RIGHTARG=bool,FUNCTION=boolge);

#132

The following statement will produce an error with pg_query_parse
while being accepted by PG13.

CREATE OPERATOR #? (LEFTARG=bool,RIGHTARG=bool,FUNCTION=boolge);

pganalyze#132
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.

👍 Thanks for the contribution!

@lfittl lfittl merged commit 9aad9fd into pganalyze:13-latest Apr 20, 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