Github user andrewor14 commented on the pull request:

    https://github.com/apache/spark/pull/5536#issuecomment-94593114
  
    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.


---
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]

Reply via email to