Fix DDL generation in case of an empty arguments function.#1690
Merged
iffyio merged 1 commit intoapache:mainfrom Jan 30, 2025
Merged
Fix DDL generation in case of an empty arguments function.#1690iffyio merged 1 commit intoapache:mainfrom
iffyio merged 1 commit intoapache:mainfrom
Conversation
remysaissy
changed the title
remysaissy
force-pushed
the
postgres/support_no_arg_functions
branch
from
3396449 to
4636691
Compare
iffyio
approved these changes
Jan 30, 2025
Contributor
iffyio
left a comment
iffyio
left a comment
There was a problem hiding this comment.
Thanks @remysaissy!
cc @alamb
iffyio
added a commit
to validio-io/sqlparser-rs
that referenced
this pull request
Jan 30, 2025
Fixes the test failure in apache#1690
Vedin
pushed a commit
to Embucket/datafusion-sqlparser-rs
that referenced
this pull request
Feb 3, 2025
Vedin
pushed a commit
to Embucket/datafusion-sqlparser-rs
that referenced
this pull request
Feb 3, 2025
Vedin
added a commit
to Embucket/datafusion-sqlparser-rs
that referenced
this pull request
Feb 3, 2025
…pache#1690)" This reverts commit 6f29ca2.
ayman-sigma
pushed a commit
to sigmacomputing/sqlparser-rs
that referenced
this pull request
Apr 10, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Functions with an empty argument list are properly parsed into the AST but the string representation of such function removes the parenthesis required after the function name, causing an error at the database level.
For eg.
CREATE OR REPLACE FUNCTION no_arg() RETURNS VOID LANGUAGE plpgsql AS $$ BEGIN DELETE FROM my_table; END; $$Becomes:
CREATE OR REPLACE FUNCTION no_arg RETURNS VOID LANGUAGE plpgsql AS $$ BEGIN DELETE FROM my_table; END; $$And causes Postgres to reject the query with a
[96579] ERROR: syntax error at or near "RETURNS"This PR fixes the string representation in this specific case.