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

New catalog can't drop functions with table arguments #539

Closed
maxhawkins opened this issue Jun 7, 2020 · 0 comments · Fixed by #541
Closed

New catalog can't drop functions with table arguments #539

maxhawkins opened this issue Jun 7, 2020 · 0 comments · Fixed by #541

Comments

@maxhawkins
Copy link
Contributor

maxhawkins commented Jun 7, 2020

CREATE FUNCTION f(x INT) RETURNS TABLE (y INT) AS 'SELECT 1' LANGUAGE sql;
DROP FUNCTION f(INT);

On the experimental parser this returns relation "f" does not exist.

This is because f() has a table argument y INT that is tripping up DROP FUNCTION. The catalog needs to be updated to detect table parameters and not use them when finding a function signature.

@maxhawkins maxhawkins changed the title New parser interprets function's table return as an argument New catalog can't drop functions with output arguments Jun 7, 2020
@maxhawkins maxhawkins changed the title New catalog can't drop functions with output arguments New catalog can't drop functions with table arguments Jun 7, 2020
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 a pull request may close this issue.

1 participant