You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2021/12/10 17:39:53 UTC

[GitHub] [spark] xkrogen commented on pull request #34856: [SPARK-37602][CORE] Add config property to set default Spark listeners

xkrogen commented on pull request #34856:
URL: https://github.com/apache/spark/pull/34856#issuecomment-991165833


   Thanks for posting this Shardul! I wish we had a more nicely unified mechanism for defining these kind of "default"/"admin" configs (e.g. the config system automatically understanding a suffix of ".default"), but at least `withPrepended` makes the implementation simple.


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