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

ASF GitHub Bot commented on TAJO-1112:
--------------------------------------

Github user hyunsik commented on the pull request:

    https://github.com/apache/tajo/pull/200#issuecomment-59467650
  
    Hi @mvhlong, 
    
    Why don't you make a separate histogram implementation for each type? We 
may need only three implementations for Long, Double, and byte []. They will 
cover all value types. I think that this approach would be good in practice.
    
    I have another question. Your proposed histogram implementation takes a 
list of Double values. Does this approach takes all values at a time? 
Otherwise, can we build a histogram in an incremental way?


> Implement histogram interface and a candidate histogram
> -------------------------------------------------------
>
>                 Key: TAJO-1112
>                 URL: https://issues.apache.org/jira/browse/TAJO-1112
>             Project: Tajo
>          Issue Type: Sub-task
>            Reporter: Long Mvh
>            Assignee: Long Mvh
>




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

Reply via email to