[
https://issues.apache.org/jira/browse/TAJO-1112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14172082#comment-14172082
]
ASF GitHub Bot commented on TAJO-1112:
--------------------------------------
Github user jihoonson commented on the pull request:
https://github.com/apache/tajo/pull/200#issuecomment-59166545
Thanks for your comment!
On using the system time function, you are right. Any unnecessary packaging
dependencies must be avoided.
On using Datum, I have a different opinion. I think that we need to extend
the histogram later to support not only columns of the numeric type, but also
columns of the text and more complex types.
However, I think the 'double' type is not sufficient to contain every type
of data.
Even though you consider only the numeric types in this issue, we can
easily extend the histogram interface later if it is designed for the more
general type, Datum.
What do you think about it?
If any other guys have other opinions about this, please feel free to
suggest!
> 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)
