Github user sryza commented on the pull request:
https://github.com/apache/spark/pull/5536#issuecomment-94595477
Mind if I take a look before you merge? I think I'm the last one to have
touched this code.
> On Apr 20, 2015, at 4:40 PM, andrewor14 <[email protected]> wrote:
>
> By the way, I plan to rewrite a large part of this logic in a way that is
more intuitive, after which any test you write here may not apply anymore so it
might make sense to hold that off for now. This code has grown to be quite
unmanageable such that even I, the original author of this feature, need to
spend a significant chunk of time trying to follow the logic and understand the
root cause of the issue.
>
> For this reason I'm going to merge this as is into master and 1.3. Thanks
@piaozhexiu @sryza @jerryshao.
>
> â
> Reply to this email directly or view it on GitHub.
>
---
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]
