Github user mateiz commented on the pull request:
https://github.com/apache/spark/pull/1009#issuecomment-45819540
Thanks Andrew for the comments. @pwendell @rxin I think this is good to
merge now if it passes tests because other people want to contribute some other
things (e.g. moving aggregation into the shuffle implementation) and I'd like
to let them do that in parallel.
---
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.
---