spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tgravescs <...@git.apache.org>
Subject [GitHub] spark pull request: [SPARK-12343][YARN] Simplify Yarn client and c...
Date Wed, 09 Mar 2016 17:03:09 GMT
Github user tgravescs commented on the pull request:

    https://github.com/apache/spark/pull/11603#issuecomment-194398314
  
    Personally I'm fine with leaving some parameters in ClientArguments as long as they don't
overlap with configs.  The main thing is to not have to process the same config/setting multiple
times.  Here we either need the parameters or add in new configs. Since these are all private
we can change it later if 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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

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


Mime
View raw message