Github user sachingoel0101 commented on the pull request:

    https://github.com/apache/flink/pull/861#issuecomment-132439907
  
    Including the collect step in the documentation seems weird, IMO. Since the 
return type is a `DataSet[Histogram]`, anyone familiar with flink will know to 
collect it first before operating on the element itself. 
    
    Of course, one other option would be actually collect and return the 
histogram in the `create` functions, but we don't wanna do that in case it is 
used inside an iteration.
    Should I include the collect step in the documentation then?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to