Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/7093#issuecomment-117829855
Heroic effort here. It certainly is big but is fixing up a lot of little
issues of this form. I flagged a few more questions about it but it is looking
pretty good.
Is anyone else concerned about the cost of future merge conflicts on this
one vs the benefit?
---
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]
