Skip to content

Commit

Permalink
[SPARK-48565][UI] Fix thread dump display in UI
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

Thread dump display in UI is not pretty as before, this is side-effect introduced by SPARK-44863

### Why are the changes needed?

Restore thread dump display in UI.

### Does this PR introduce _any_ user-facing change?

Yes, it only affects UI display.

### How was this patch tested?

Current master:
<img width="1545" alt="master-branch" src="https://github.com/apache/spark/assets/26535726/5c6fd770-467f-481c-a635-2855a2853633">

With this patch applied:
<img width="1542" alt="Xnip2024-06-07_20-00-38" src="https://github.com/apache/spark/assets/26535726/3998c2aa-671f-4921-8444-b7bca8667202">

### Was this patch authored or co-authored using generative AI tooling?

No

Closes #46916 from pan3793/SPARK-48565.

Authored-by: Cheng Pan <chengpan@apache.org>
Signed-off-by: Kent Yao <yao@apache.org>
  • Loading branch information
pan3793 authored and yaooqinn committed Jun 11, 2024
1 parent 1e4750e commit 53d65fd
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -510,7 +510,7 @@ case class StackTrace(elems: Seq[String]) {
override def toString: String = elems.mkString

def html: NodeSeq = {
val withNewLine = elems.foldLeft(NodeSeq.Empty) { (acc, elem) =>
val withNewLine = elems.map(_.stripLineEnd).foldLeft(NodeSeq.Empty) { (acc, elem) =>
if (acc.isEmpty) {
acc :+ Text(elem)
} else {
Expand Down

0 comments on commit 53d65fd

Please sign in to comment.