Github user ArtRand commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19793#discussion_r153348394
  
    --- Diff: 
resource-managers/mesos/src/main/scala/org/apache/spark/deploy/rest/mesos/MesosRestServer.scala
 ---
    @@ -82,6 +82,12 @@ private[mesos] class MesosSubmitRequestServlet(
         val mainClass = Option(request.mainClass).getOrElse {
           throw new SubmitRestMissingFieldException("Main class is missing.")
         }
    +    val appArgs = Option(request.appArgs).getOrElse {
    +      throw new SubmitRestMissingFieldException("Application arguments are 
missing.")
    --- End diff --
    
    Nit: maybe put the fields here also? Something like `Application arguments 
("appArgs") are missing`. Similar below. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to