Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/7774#issuecomment-127468953
> In terms of a way forward with this patch, I would suggest that we try to
merge the MVP version first (one that doesn't address all the comments) as soon
as possible because this patch is so large and conflict-prone. Most of the
things I pointed out in my review can be addressed later, so let's just focus
on resolving the outstanding conflicts and making it pass tests first.
I fixed minor things which should not change the logic in this PR. For
other comments, it's better to fix them in some follow-up PRs so that we can
make this PR merge quickly.
---
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]
