[ https://issues.apache.org/jira/browse/HIVE-16484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16318872#comment-16318872 ]
Sahil Takiar commented on HIVE-16484: ------------------------------------- Sounds good to me. I think release for Spark 2.3.0 is just starting - http://apache-spark-developers-list.1001551.n3.nabble.com/Branch-2-3-is-cut-td23072.html Once that gets started I'll work on using {{InProcessLauncher}} in a new {{SparkClient}}. I'm going to close this JIRA for now, the comments and patch history are already pretty long. The original intention was just an investigation, which I think is complete since we have a plan going forward. I'll work on the {{InProcessLauncher}} integration in another JIRA. > Investigate SparkLauncher for HoS as alternative to bin/spark-submit > -------------------------------------------------------------------- > > Key: HIVE-16484 > URL: https://issues.apache.org/jira/browse/HIVE-16484 > Project: Hive > Issue Type: Bug > Components: Spark > Reporter: Sahil Takiar > Assignee: Sahil Takiar > Attachments: HIVE-16484.1.patch, HIVE-16484.10.patch, > HIVE-16484.2.patch, HIVE-16484.3.patch, HIVE-16484.4.patch, > HIVE-16484.5.patch, HIVE-16484.6.patch, HIVE-16484.7.patch, > HIVE-16484.8.patch, HIVE-16484.9.patch > > > The {{SparkClientImpl#startDriver}} currently looks for the {{SPARK_HOME}} > directory and invokes the {{bin/spark-submit}} script, which spawns a > separate process to run the Spark application. > {{SparkLauncher}} was added in SPARK-4924 and is a programatic way to launch > Spark applications. > I see a few advantages: > * No need to spawn a separate process to launch a HoS --> lower startup time > * Simplifies the code in {{SparkClientImpl}} --> easier to debug > * {{SparkLauncher#startApplication}} returns a {{SparkAppHandle}} which > contains some useful utilities for querying the state of the Spark job > ** It also allows the launcher to specify a list of job listeners -- This message was sent by Atlassian JIRA (v6.4.14#64029)