Github user aarondav commented on the pull request:
https://github.com/apache/spark/pull/3378#issuecomment-63769839
This does seem like a good change, though I'll note that I think groupBy is
the only current user of this API that is able to have a primitive ClassTag.
Still worthwhile, especially for future usage. I do wonder if it could have a
runtime impact due to increased primitive wrapping, possibly creating a lot of
short-lived garbage if it were iterated over many times.
---
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.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
