Github user susanxhuynh commented on a diff in the pull request: https://github.com/apache/spark/pull/19793#discussion_r153352732 --- Diff: core/src/test/scala/org/apache/spark/deploy/rest/SubmitRestProtocolSuite.scala --- @@ -86,6 +86,8 @@ class SubmitRestProtocolSuite extends SparkFunSuite { message.clientSparkVersion = "1.2.3" message.appResource = "honey-walnut-cherry.jar" message.mainClass = "org.apache.spark.examples.SparkPie" + message.appArgs = Array("hdfs://tmp/auth") + message.environmentVariables = Map("SPARK_HOME" -> "/test") --- End diff -- There should be a check at the end of this test to make sure these fields show up in the `newMessage` object. (around L.125)
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org