ConeyLiu edited a comment on pull request #29770: URL: https://github.com/apache/spark/pull/29770#issuecomment-693414677
Hi @tgravescs. Thanks for the advice. I have not tested all arguments, however, it works for us now. I agree with you, we need to redesign the `ExternalClusterManager ` to support full arguments. Actually, the special logic for each `ClusterManager` should be processed in each `ClusterManager`, not in the `SparkSubmit`. The current change should be the smallest change. cc @carsonwang ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
