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

UDAFs do not respect the Configurable interface #5738

Closed
MichaelDrogalis opened this issue Jun 30, 2020 · 1 comment · Fixed by #6851
Closed

UDAFs do not respect the Configurable interface #5738

MichaelDrogalis opened this issue Jun 30, 2020 · 1 comment · Fixed by #6851
Labels

Comments

@MichaelDrogalis
Copy link
Contributor

Describe the bug

Version: 0.10.0, related to #5737

The docs describe that a UDF can be made configurable with external parameters by implementing the Configurable interface. This works as described for scalar functions. But it doesn't work for UDAFs. The configure() method is silently ignored. This might be a bit worse than #5737 because UDAF factories expect to return a Udaf or TableUdaf. Because Java doesn't support type unions, we might need to jump through some hoops to get ksqlDB to realize which UDAFs are configurable and which are not.

Either way, this behavior should also function for UDAFs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants