[GitHub] [spark] attilapiros commented on pull request #41746: [SPARK-44198][CORE] Support propagation of the log level to the executors

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

attilapiros commented on PR #41746:
URL: https://github.com/apache/spark/pull/41746#issuecomment-1631577518

   I do not like the idea of using a spark config for a dynamically changing value like the updated log level. 
   
   So I would store the the updated log level in `CoarseGrainedSchedulerBackend` in a member var (`Option[String]`) update it whenever a `UpdateExecutorsLogLevel` arrives and pass that to the executor backends in a new member val of the `SparkAppConfig`. 
   
   
   
   


-- 
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