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

Fabian Hueske commented on FLINK-3657:
--------------------------------------

As I said before, I don't think this functionality is too specific since the 
Mahout community was asking for it. IMO it could be added to DataSetUtils. I 
was only raising the question whether this should be done as part of a bugfix 
release (1.0.1) or not. From my point of view there were arguments for both 
decisions. 
+ the change is minor, not touching the core API, and requested for a Mahout 
release with Flink support.
- Flink 1.0.1 would not be a "clean" bugfix release and it could also be 
implemented as part of Mahout

> Change access of DataSetUtils.countElements() to 'public' 
> ----------------------------------------------------------
>
>                 Key: FLINK-3657
>                 URL: https://issues.apache.org/jira/browse/FLINK-3657
>             Project: Flink
>          Issue Type: Improvement
>          Components: DataSet API
>    Affects Versions: 1.0.0
>            Reporter: Suneel Marthi
>            Assignee: Suneel Marthi
>            Priority: Minor
>             Fix For: 1.0.1
>
>
> The access of DatasetUtils.countElements() is presently 'private', change 
> that to be 'public'. We happened to be replicating the functionality in our 
> project and realized the method already existed in Flink.



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

Reply via email to