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

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

Github user chiwanpark commented on the pull request:

    https://github.com/apache/flink/pull/861#issuecomment-137912454
  
    I object to break API in this PR. I think that this PR should cover 
histogram for `DataSet<Double>` only. That's why we split this PR from #1032. 
About documentation, adding a document file such as `building_histogram.md` 
first and merging it later would be good.
    
    For Scala API, the current implementation looks good for me.


> Implement discrete and continuous histograms
> --------------------------------------------
>
>                 Key: FLINK-2030
>                 URL: https://issues.apache.org/jira/browse/FLINK-2030
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Core
>            Reporter: Sachin Goel
>            Assignee: Sachin Goel
>            Priority: Minor
>
> For the implementation of the decision tree in 
> https://issues.apache.org/jira/browse/FLINK-1727, we need to implement an 
> histogram with online updates, merging and equalization features. A reference 
> implementation is provided in [1]
> [1].http://www.jmlr.org/papers/volume11/ben-haim10a/ben-haim10a.pdf



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

Reply via email to