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

Stephan Ewen commented on FLINK-4330:
-------------------------------------

I the {{DataSet}} API, these methods made sense. There was also a clear 
distinction between what {{max()}} and {{maxBy()}} does.
This does not work as well in the {{DataStream}} API due to some missing 
features.

I think we should deprecate and remove them by the time we have a suitable 
replacement via the Table API.

> Consider removing min()/minBy()/max()/maxBy()/sum() utility methods from the 
> DataStream API
> -------------------------------------------------------------------------------------------
>
>                 Key: FLINK-4330
>                 URL: https://issues.apache.org/jira/browse/FLINK-4330
>             Project: Flink
>          Issue Type: Sub-task
>          Components: DataStream API
>            Reporter: Robert Metzger
>             Fix For: 2.0.0
>
>
> I think we should consider removing the min()/minBy()/max()/maxBy()/sum() 
> utility methods from the DataStream API. They make the maintenance of the 
> code unnecessary complicated, and don't add enough value for the users (who 
> can not access the window metadata).
> If we are keeping the methods, we should consolidate the min/minBy methods: 
> the difference is subtle, and minBy can subsume the min method.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to