Skip to content

Commit

Permalink
[SPARK-37860][UI] Fix taskindex in the stage page task event timeline
Browse files Browse the repository at this point in the history
### 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]>
(cherry picked from commit 3d2fde5)
Signed-off-by: Kousuke Saruta <[email protected]>
  • Loading branch information
jackylee-ch authored and sarutak committed Jan 11, 2022
1 parent 1a27539 commit 830d5b6
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,7 @@ private[ui] class StagePage(parent: StagesTab, store: AppStatusStore) extends We
|'content': '<div class="task-assignment-timeline-content"
|data-toggle="tooltip" data-placement="top"
|data-html="true" data-container="body"
|data-title="${s"Task " + taskInfo.taskId + " (attempt " + attempt + ")"}<br>
|data-title="${s"Task " + index + " (attempt " + attempt + ")"}<br>
|Status: ${taskInfo.status}<br>
|Launch Time: ${UIUtils.formatDate(new Date(launchTime))}
|${
Expand Down

0 comments on commit 830d5b6

Please sign in to comment.