Repository: spark Updated Branches: refs/heads/branch-2.1 5bafdc45d -> ca25b1e51
[SPARK-18837][WEBUI] Very long stage descriptions do not wrap in the UI ## What changes were proposed in this pull request? This issue was reported by wangyum. In the AllJobsPage, JobPage and StagePage, the description length was limited before like as follows.  But recently, the limitation seems to have been accidentally removed.  The cause is that some tables are no longer `sortable` class although they were, and `sortable` class does not only mark tables as sortable but also limited the width of their child `td` elements. The reason why now some tables are not `sortable` class is because another sortable mechanism was introduced by #13620 and #13708 with pagination feature. To fix this issue, I've introduced new class `table-cell-width-limited` which limits the description cell width and the description is like what it was. <img width="1260" alt="2016-12-20 1 00 34" src="https://cloud.githubusercontent.com/assets/4736016/21320478/89141c7a-c654-11e6-8494-f8f91325980b.png";> ## How was this patch tested? Tested manually with my browser. Author: Kousuke Saruta <saru...@oss.nttdata.co.jp> Closes #16338 from sarutak/SPARK-18837. (cherry picked from commit f2ceb2abe9357942a51bd643683850efd1fc9df7) Signed-off-by: Sean Owen <so...@cloudera.com> Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/ca25b1e5 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/ca25b1e5 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/ca25b1e5 Branch: refs/heads/branch-2.1 Commit: ca25b1e51f036fb837e3fe8218cb04d7360e049d Parents: 5bafdc4 Author: Kousuke Saruta <saru...@oss.nttdata.co.jp> Authored: Sat Dec 24 13:02:58 2016 +0000 Committer: Sean Owen <so...@cloudera.com> Committed: Sat Dec 24 13:03:07 2016 +0000 ---------------------------------------------------------------------- core/src/main/resources/org/apache/spark/ui/static/webui.css | 4 ++++ core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala | 3 ++- core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala | 3 ++- 3 files changed, 8 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/spark/blob/ca25b1e5/core/src/main/resources/org/apache/spark/ui/static/webui.css ---------------------------------------------------------------------- diff --git a/core/src/main/resources/org/apache/spark/ui/static/webui.css b/core/src/main/resources/org/apache/spark/ui/static/webui.css index b157f3e..319a719 100644 --- a/core/src/main/resources/org/apache/spark/ui/static/webui.css +++ b/core/src/main/resources/org/apache/spark/ui/static/webui.css @@ -246,4 +246,8 @@ a.expandbutton { text-align: center; margin: 0; padding: 4px 0; +} + +.table-cell-width-limited td { + max-width: 600px; } \ No newline at end of file http://git-wip-us.apache.org/repos/asf/spark/blob/ca25b1e5/core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala ---------------------------------------------------------------------- diff --git a/core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala b/core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala index 173fc3c..d9475c4 100644 --- a/core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala +++ b/core/src/main/scala/org/apache/spark/ui/jobs/AllJobsPage.scala @@ -500,7 +500,8 @@ private[ui] class JobPagedTable( override def tableId: String = jobTag + "-table" override def tableCssClass: String = - "table table-bordered table-condensed table-striped table-head-clickable" + "table table-bordered table-condensed table-striped " + + "table-head-clickable table-cell-width-limited" override def pageSizeFormField: String = jobTag + ".pageSize" http://git-wip-us.apache.org/repos/asf/spark/blob/ca25b1e5/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala ---------------------------------------------------------------------- diff --git a/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala b/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala index c9d0431..e1fa904 100644 --- a/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala +++ b/core/src/main/scala/org/apache/spark/ui/jobs/StageTable.scala @@ -149,7 +149,8 @@ private[ui] class StagePagedTable( override def tableId: String = stageTag + "-table" override def tableCssClass: String = - "table table-bordered table-condensed table-striped table-head-clickable" + "table table-bordered table-condensed table-striped " + + "table-head-clickable table-cell-width-limited" override def pageSizeFormField: String = stageTag + ".pageSize" --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org For additional commands, e-mail: commits-h...@spark.apache.org
![https://cloud.githubusercontent.com/assets/4736016/21320478/89141c7a-c654-11e6-8494-f8f91325980b.png"](https://cloud.githubusercontent.com/assets/4736016/21320478/89141c7a-c654-11e6-8494-f8f91325980b.png"){kind=link}