otterc opened a new pull request, #40448:
URL: https://github.com/apache/spark/pull/40448

   ### What changes were proposed in this pull request?
   Removed the logging of shuffle service name multiple times in the driver 
log. It gets logged everytime a new executor is allocated.
   
   ### Why are the changes needed?
   This is needed because currently the driver logs gets polluted by these logs:
   ```
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   22/08/03 20:42:07 INFO ExecutorRunnable: Initializing service data for 
shuffle service using name 'spark_shuffle_311'
   ```
   
   ### Does this PR introduce _any_ user-facing change?
   Yes, the shuffle service name will be just logged once in the driver.
   
   
   ### How was this patch tested?
   Tested manually since it just changes the logging.
   With this see this logged in the driver logs:
   `23/03/15 16:50:54 INFO  ApplicationMaster: Initializing service data for 
shuffle service using name 'spark_shuffle_311'`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to