[GitHub] [spark] robreeves commented on pull request #40637: [SPARK-43002][YARN] Modify yarn client application report logging frequency to reduce noise

["robreeves (via GitHub)" <gi...@apache.org>]

robreeves commented on PR #40637:
URL: https://github.com/apache/spark/pull/40637#issuecomment-1498070306

   > how is this different from:
   >
   >  private[spark] val REPORT_INTERVAL = ConfigBuilder("spark.yarn.report.interval")
   >   .doc("Interval between reports of the current app status.")
   >    .version("0.9.0")
   >    .timeConf(TimeUnit.MILLISECONDS)
   >    .createWithDefaultString("1s")
   >
   > ?
   
   @tgravescs The new property allows for the ability to reduce the log noise without reducing the responsiveness when something changes because the status check interval is not reduced like if `spark.yarn.report.interval` was used alone. For example, if a job failed right away waiting 30 seconds to get that feedback slows down the iteration cycle.


-- 
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-unsubscribe@spark.apache.org

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


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