Github user ajithme commented on the issue:

    https://github.com/apache/spark/pull/22120
  
    @vanzin I agree its a trivial change. Just wanted it to be consistent 
output with yarn cluster mode. This is not just for event logs also for a 
custom SparkListener , it may be confusing that appId is empty in client case 
and a actual number in cluster case for onApplicationStart, this is where its 
effect can be seen. 


---

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

Reply via email to