Github user echarles commented on the issue:

    https://github.com/apache/spark/pull/20451
  
    @liyinan926 I pushed the merge with master and tested OutCluster (I didn't 
test InCluster but as mentioned previously, we would need something similar to 
the `DriverServiceBootstrapStep` of the fork).
    
    @squito @tnachen Thx for your reviews and comments which make sense. My 
approach is to change what is needed for the client-mode. I prefer leaving 
constructs, code removal and enhancement considerations in other PRs.


---

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

Reply via email to