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!
---
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.
---
