Ngone51 commented on a change in pull request #25037: [SPARK-27489][WEBUI] UI
updates to show executor resource information
URL: https://github.com/apache/spark/pull/25037#discussion_r304992934
##########
File path: core/src/main/resources/org/apache/spark/ui/static/executorspage.js
##########
@@ -39,6 +39,19 @@ function formatStatus(status, type, row) {
return "Dead"
}
+function formatResourceCells(resources) {
Review comment:
It seems a little crowded from the screenshot. It would be better if we
could split this single resource cell into several row cells by resource type.
But I guess this may require more changes ? Maybe, add `\n` after each resource
type would be good enough ? Or you're intentionally to do this to occupy less
space of the cell(since it brings more unnecessary space for other
columns(looks more sparse now) ) ?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
