Github user srowen commented on the pull request:

    https://github.com/apache/spark/pull/3863#issuecomment-76080519
  
    Don't worry about Jenkins tests; as I was just reminded a) Jenkins doesn't 
test the doc build and b) we'd have to manually check that it builds before 
merging. Assume it's OK for now.
    
    I did a sanity check to find the props that were removed and added to make 
sure they match up:
    
    ```
    < <code>spark.akka.failure-detector.threshold</code>
    21c20
    < <code>spark.dynamicAllocation.initialExecutors</code>
    ---
    > <code>spark.dynamicAllocation.maxExecutors</code>
    45a45,47
    > <code>spark.mesos.coarse</code>
    > <code>spark.mesos.executor.home</code>
    > <code>spark.mesos.executor.memoryOverhead</code>
    ```
    
    I think it's expected that the mesos ones are removed but what about the 
others?


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