Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10554#issuecomment-168977430
@rxin OK I _almost_ did that. I realized that `JavaRDD.countByValue`
already does a `mapValues`. I left `countByKey` to act the same way, doing the
mapping. Other methods that return `JavaPairRDD` went back to doing a cast.
(Although similar methods in the streaming API actually also do a `mapValues`.)
I also tried to use `JLong` vs `jl.Long` consistently within a file without
going overboard; both of these files use about every different form. I think
the result is more consistent internally, but WDYT? I'm neutral, and would
further change things while keeping them consistent if anyone had a preference.
---
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]
