[spark] branch master updated: [SPARK-33145][WEBUI] Fix when `Succeeded Jobs` has many child url elements, they will extend over the edge of the page

[gengliang]

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

gengliang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new e029e89  [SPARK-33145][WEBUI] Fix when `Succeeded Jobs` has many child 
url elements,they will extend over the edge of the page
e029e89 is described below

commit e029e891abeb37f383e4d5237edf693c8ad53bed
Author: neko <echoh...@gmail.com>
AuthorDate: Fri Oct 16 23:13:22 2020 +0800

    [SPARK-33145][WEBUI] Fix when `Succeeded Jobs` has many child url 
elements,they will extend over the edge of the page
    
    ### What changes were proposed in this pull request?
    In Execution web page, when `Succeeded Job`(or Failed Jobs) has many child 
url elements,they will extend over the edge of the page.
    
    ### Why are the changes needed?
    To make the page more friendly.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    
    Munual test result shows  as below:
    
    
![fixed](https://user-images.githubusercontent.com/52202080/95977319-50734600-0e4b-11eb-93c0-b8deb565bcd8.png)
    
    Closes #30035 from akiyamaneko/sql_execution_job_overflow.
    
    Authored-by: neko <echoh...@gmail.com>
    Signed-off-by: Gengliang Wang <gengliang.w...@databricks.com>
---
 .../org/apache/spark/sql/execution/ui/static/spark-sql-viz.css       | 5 +++++
 .../main/scala/org/apache/spark/sql/execution/ui/ExecutionPage.scala | 2 +-
 2 files changed, 6 insertions(+), 1 deletion(-)

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 9a32b79..dbdbf9f 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
@@ -52,3 +52,8 @@
 .tooltip-inner {
   word-wrap:break-word;
 }
+
+/* Breaks the long job url list when showing Details for Query in SQL */
+.job-url {
+  word-wrap: break-word;
+}
diff --git 
a/sql/core/src/main/scala/org/apache/spark/sql/execution/ui/ExecutionPage.scala 
b/sql/core/src/main/scala/org/apache/spark/sql/execution/ui/ExecutionPage.scala
index 76bc7fa..b15c70a 100644
--- 
a/sql/core/src/main/scala/org/apache/spark/sql/execution/ui/ExecutionPage.scala
+++ 
b/sql/core/src/main/scala/org/apache/spark/sql/execution/ui/ExecutionPage.scala
@@ -45,7 +45,7 @@ class ExecutionPage(parent: SQLTab) extends 
WebUIPage("execution") with Logging
           if (jobStatus == status) Some(jobId) else None
         }
         if (jobs.nonEmpty) {
-          <li>
+          <li class="job-url">
             <strong>{label} </strong>
             {jobs.toSeq.sorted.map { jobId =>
               <a href={jobURL(request, 
jobId.intValue())}>{jobId.toString}</a><span>&nbsp;</span>


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