Github user marmbrus commented on the pull request:
https://github.com/apache/spark/pull/9406#issuecomment-154581656
Okay, I looked over this pretty quickly and it looks awesome. We need some
tests and we are super close to me cutting a preview release. That said, I'd
really like to include this in 1.6. Here is my proposal:
- Lets merge this as is.
- Yin will start ripping out the old aggregation path. (with or without
#9409).
- Comments and TODOs can be addressed in a follow up.
---
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]
