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

    https://github.com/apache/spark/pull/23189#discussion_r237972852
  
    --- Diff: core/src/main/scala/org/apache/spark/deploy/SparkSubmit.scala ---
    @@ -813,14 +813,14 @@ private[spark] class SparkSubmit extends Logging {
           mainClass = Utils.classForName(childMainClass)
         } catch {
           case e: ClassNotFoundException =>
    -        logWarning(s"Failed to load $childMainClass.", e)
    +        logError(s"Failed to load class $childMainClass.")
    --- End diff --
    
    Here I tried append `e.getMessage` to the output, but it seems the error 
message equals  the class name `childMainClass`. e.g.
    ```
    Error: Failed to load class foo: foo.
    ```
    So I think we can output without the exception message.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to