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

Reply via email to