Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/22146
I think we still need some consensus on the driver container API.
Personally I think it's clearer to use a configuration option to specifically
select the container that is the driver, but this is an opinion I hold loosely.
Would like specific weigh-in from @erikerlandson and @liyinan926.
Aside from that, this looks good on my end apart from the tests we need to
add and should be ready to merge pretty soon (depending also on Spark 2.4
timeline)
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
