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

Argument count fixed for binary custom functions. #481

Closed
wants to merge 2 commits into from
Closed

Argument count fixed for binary custom functions. #481

wants to merge 2 commits into from

Conversation

andrei-v-frolov
Copy link

@andrei-v-frolov andrei-v-frolov commented Jul 29, 2016 •
edited
Loading

Looks like there is a trivial bug in custom SQL function bindings for functions with two arguments.

@jberkel
Copy link
Collaborator

jberkel commented Sep 15, 2016 •
edited
Loading

Thanks for your contribution. Can you add some test cases for your changes?

@andrei-v-frolov
Copy link
Author

Will do.

@jberkel jberkel mentioned this pull request Sep 20, 2016
Closed
@jberkel jberkel added this to the 0.11.2 milestone Dec 6, 2016
@jberkel jberkel modified the milestones: 0.11.2, 0.11.3 Dec 29, 2016
jberkel added a commit that referenced this pull request Mar 15, 2017
@jberkel
Argument count fixed for binary custom functions
47e109a 
Closes #481
@jberkel
Copy link
Collaborator

jberkel commented Mar 15, 2017

Part of #616

@jberkel jberkel closed this Mar 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.11.3
Development

Successfully merging this pull request may close these issues.
2 participants
@andrei-v-frolov @jberkel