You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by rvesse <gi...@git.apache.org> on 2018/01/15 15:09:48 UTC

[GitHub] spark issue #20167: [SPARK-16501] [MESOS] Allow providing Mesos principal & ...

Github user rvesse commented on the issue:

    https://github.com/apache/spark/pull/20167
  
    @vanzin @ArtRand Thanks for the initial reviews, I have refactored the patch based on comments so should hopefully be in a better state now.  I was able to also add some unit test coverage for this functionality.
    
    I will look at also updating documentation appropriately which will need to be a separate PR to the apache/spark-website repository


---

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