Skip to content

Commit 3d2fde5

Browse files
stczwdsarutak
stczwd
authored andcommitted
[SPARK-37860][UI] Fix taskindex in the stage page task event timeline
### What changes were proposed in this pull request? This reverts commit 450b415. ### Why are the changes needed? In #32888, shahidki31 change taskInfo.index to taskInfo.taskId. However, we generally use `index.attempt` or `taskId` to distinguish tasks within a stage, not `taskId.attempt`. Thus #32888 was a wrong fix issue, we should revert it. ### Does this PR introduce _any_ user-facing change? no ### How was this patch tested? origin test suites Closes #35160 from stczwd/SPARK-37860. Authored-by: stczwd <[email protected]> Signed-off-by: Kousuke Saruta <[email protected]>
1 parent 7463564 commit 3d2fde5

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -355,7 +355,7 @@ private[ui] class StagePage(parent: StagesTab, store: AppStatusStore) extends We
355355
|'content': '<div class="task-assignment-timeline-content"
356356
|data-toggle="tooltip" data-placement="top"
357357
|data-html="true" data-container="body"
358-
|data-title="${s"Task " + taskInfo.taskId + " (attempt " + attempt + ")"}<br>
358+
|data-title="${s"Task " + index + " (attempt " + attempt + ")"}<br>
359359
|Status: ${taskInfo.status}<br>
360360
|Launch Time: ${UIUtils.formatDate(new Date(launchTime))}
361361
|${

0 commit comments

Comments
 (0)