[ 
https://issues.apache.org/jira/browse/FLINK-6457?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16689514#comment-16689514
 ] 

ASF GitHub Bot commented on FLINK-6457:
---------------------------------------

twalthr commented on issue #3880: [FLINK-6457] [table] Clean up ScalarFunction 
and TableFunction interface
URL: https://github.com/apache/flink/pull/3880#issuecomment-439417094
 
 
   I also agree with @fhueske. I will close this PR for now. The affected 
classes might be ported to Java soon anyway due to FLINK-10689.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


> Clean up ScalarFunction and TableFunction interface
> ---------------------------------------------------
>
>                 Key: FLINK-6457
>                 URL: https://issues.apache.org/jira/browse/FLINK-6457
>             Project: Flink
>          Issue Type: Improvement
>          Components: Table API & SQL
>            Reporter: Ruidong Li
>            Assignee: Ruidong Li
>            Priority: Major
>              Labels: pull-request-available
>
> Motivation:
> Some methods in ScalarFunction and TableFunction are unnecessary, e.g 
> toString() and getResultType in ScalarFunction
> this issue intend to clear the interface.
> Goal:
>  only methods related to `Collector` will remain in TableFunction interface, 
> and ScalarFunction interface shall have no methods , user can choose whether 
> to implement the `getResultType` method, which will be called by reflection, 
> and the Flink document will have instructions for user.
> Future:
> There should be some Annotations for user to implement methods like `@Eval` 
> for eval method, it be will in the next issue



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to