spark-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kayousterhout <...@git.apache.org>
Subject [GitHub] spark pull request: [Proposal] SPARK-1171: simplify the implementa...
Date Tue, 04 Mar 2014 17:54:10 GMT
Github user kayousterhout commented on the pull request:

    https://github.com/apache/spark/pull/63#issuecomment-36653243
  
    This new version of the change doesn't look any simpler to me than the current version
of the code and I think is a slightly confusing way of using worker offers to store info about
the executors. Can you just remove executorAddress, the unused variable, and fix the bug,
but keep the original way of generating worker offers?


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

Mime
View raw message