You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by do...@apache.org on 2020/04/26 23:59:02 UTC

[spark] branch branch-3.0 updated: [SPARK-31565][WEBUI] Unify the font color of label among all DAG-viz

This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new a602ab6  [SPARK-31565][WEBUI] Unify the font color of label among all DAG-viz
a602ab6 is described below

commit a602ab6c1a2f5d2e47970c9f64c62d4e9500dce6
Author: Kousuke Saruta <sa...@oss.nttdata.com>
AuthorDate: Sun Apr 26 16:57:23 2020 -0700

    [SPARK-31565][WEBUI] Unify the font color of label among all DAG-viz
    
    ### What changes were proposed in this pull request?
    
    This PR unifies the font color of label as `#333333` among all DAG-viz.
    
    ### Why are the changes needed?
    
    For the consistent appearance among all DAG-viz.
    There are three types of DAG-viz in the WebUI.
    One is for stages, another one is for RDDs and the last one is for query plans.
    But the font color of labels are slightly different among them.
    
    For stages, the color is `#333333` (simply 333) which is specified by `spark-dag-viz.css`.
    <img width="355" alt="job-graph" src="https://user-images.githubusercontent.com/4736016/80321397-b517f580-8857-11ea-8c8e-cf68f648ab05.png">
    <img width="310" alt="job-graph-color" src="https://user-images.githubusercontent.com/4736016/80321399-ba754000-8857-11ea-8708-83bdef4bc1d1.png">
    
    For RDDs, the color is `#212529` which is specified by `bootstrap.min.js`.
    <img width="386" alt="stage-graph" src="https://user-images.githubusercontent.com/4736016/80321438-f0b2bf80-8857-11ea-9c2a-13fa0fd1431c.png">
    <img width="313" alt="stage-graph-color" src="https://user-images.githubusercontent.com/4736016/80321444-fa3c2780-8857-11ea-81b2-4f1203d47896.png">
    
    For query plans, the color is `black` which is specified by `spark-sql-viz.css`.
    <img width="449" alt="plan-graph" src="https://user-images.githubusercontent.com/4736016/80321490-61f27280-8858-11ea-9c3a-2c98d3d4d03b.png">
    <img width="316" alt="plan-graph-color" src="https://user-images.githubusercontent.com/4736016/80321496-6ae34400-8858-11ea-8fe8-0d6e4a821608.png">
    
    After the change, the appearance is like as follows (no change for stages).
    
    For RDDs.
    <img width="389" alt="stage-graph-fixed" src="https://user-images.githubusercontent.com/4736016/80321613-6b300f00-8859-11ea-912f-d92474aa9f47.png">
    
    For query plans.
    <img width="456" alt="plan-graph-fixed" src="https://user-images.githubusercontent.com/4736016/80321638-9a468080-8859-11ea-974c-33c56a8ffe1a.png">
    
    ### Does this PR introduce any user-facing change?
    
    Yes. The unified color is slightly lighter than ever.
    
    ### How was this patch tested?
    
    Confirmed that the color code among all DAG-viz are `#333333` using browser's debug console.
    
    Closes #28352 from sarutak/unify-label-color.
    
    Authored-by: Kousuke Saruta <sa...@oss.nttdata.com>
    Signed-off-by: Dongjoon Hyun <do...@apache.org>
    (cherry picked from commit 91ec2eacfaa030c7767bf1d192d5db8f56ff6854)
    Signed-off-by: Dongjoon Hyun <do...@apache.org>
---
 core/src/main/resources/org/apache/spark/ui/static/spark-dag-viz.css    | 1 +
 .../org/apache/spark/sql/execution/ui/static/spark-sql-viz.css          | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/core/src/main/resources/org/apache/spark/ui/static/spark-dag-viz.css b/core/src/main/resources/org/apache/spark/ui/static/spark-dag-viz.css
index 1fbc90b..e44a724 100644
--- a/core/src/main/resources/org/apache/spark/ui/static/spark-dag-viz.css
+++ b/core/src/main/resources/org/apache/spark/ui/static/spark-dag-viz.css
@@ -22,6 +22,7 @@
 #dag-viz-graph .label {
   font-weight: normal;
   text-shadow: none;
+  color: #333;
 }
 
 #dag-viz-graph svg path {
diff --git a/sql/core/src/main/resources/org/apache/spark/sql/execution/ui/static/spark-sql-viz.css b/sql/core/src/main/resources/org/apache/spark/sql/execution/ui/static/spark-sql-viz.css
index ffc9acd..25198db 100644
--- a/sql/core/src/main/resources/org/apache/spark/sql/execution/ui/static/spark-sql-viz.css
+++ b/sql/core/src/main/resources/org/apache/spark/sql/execution/ui/static/spark-sql-viz.css
@@ -18,7 +18,7 @@
 #plan-viz-graph .label {
   font-weight: normal;
   text-shadow: none;
-  color: black;
+  color: #333;
 }
 
 #plan-viz-graph svg g.cluster rect {


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@spark.apache.org
For additional commands, e-mail: commits-help@spark.apache.org