Github user jaceklaskowski commented on the pull request:

    https://github.com/apache/spark/pull/8976#issuecomment-146474747
  
    Fully agree. I'm however not skilled to work on the feature to enable 
locality preference and haven't heard about anyone working on it, and the code 
as it is now is *not* used and is misleading. That said, I'd remove it and 
bring back when it's needed (the comments do more harm than offer help).
    
    BTW, It's all versioned in the repo and one can easily revert to any 
version in the past when needed.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to