Github user markhamstra commented on the pull request:
https://github.com/apache/spark/pull/893#issuecomment-44309000
@ash211 Makes sense to me -- which doesn't necessarily mean a lot in this
unfamiliar area of the code.... It looks to me like the dataType for each of
CountDistinct, ApproxCountDistinctMerge and ApproxCountDistinct should be
LongType.
And while we are in this file, AggregateFunction#eval(input: Row): Any
doesn't actually override anything and looks to me to be superfluous.
---
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.
---
