hai-tao-1 commented on code in PR #36606: URL: https://github.com/apache/spark/pull/36606#discussion_r878786954
########## core/src/main/scala/org/apache/spark/internal/config/package.scala: ########## @@ -2363,4 +2363,12 @@ package object config { .version("3.3.0") .intConf .createWithDefault(5) + + val SPARK_PROPERTIES_KEY_QUEUE = ConfigBuilder("spark.properties.key.queue") Review Comment: > I'm not sure about adding an empty column to UI to all resource managers. This had better be disabled by default. > > cc @gengliangwang too because this is a breaking UI change. Even though I believe Queue information is very common in the platforms running Spark, we could start with only showing the queue column if the configuration `spark.properties.key.queue` is specified. -- 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