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.

Sign up for GitHub

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

Jump to bottom

Failure to parse SqlFunctionHandle #24077

Open
pdabre12 opened this issue Nov 18, 2024 · 0 comments · May be fixed by #24059
Open

Failure to parse SqlFunctionHandle #24077

pdabre12 opened this issue Nov 18, 2024 · 0 comments · May be fixed by #24059
Assignees
@pdabre12
Labels
bug prestissimo Presto Native Execution

Comments

@pdabre12
Copy link
Contributor

Unable to parse SqlFunctionHandle when the functions passed down have zero params or greater than one param.

Your Environment

  • Presto version used: Latest master
  • Storage (HDFS/S3/GCS..):
  • Data source and connector used:
  • Deployment (Cloud or On-prem):
  • Pastebin link to the complete debug logs:

Expected Behavior

The params in the SqlFunctionHandle should be parsed correctly.

Current Behavior

While parsing the SqlFunctionHandle, the exit condition never gets hit leading to a never-ending loop.

Possible Solution

A fix: #24059

Steps to Reproduce

Unit tests included in PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pdabre12 pdabre12

Labels
bug prestissimo Presto Native Execution
Projects
Bugs and support requests
Status: 🆕 Unprioritized
Prestissimo Gap
Status: Backlog
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[native] Fix bug when parsing SqlFunctionHandle pdabre12/presto
1 participant
@pdabre12