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

sarutak pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new 6597c3b  [SPARK-35746][UI] Fix taskid in the stage page task event 
timeline
6597c3b is described below

commit 6597c3bd5e91040dc53576c912c85d84f630bb17
Author: shahid <shahidk...@gmail.com>
AuthorDate: Sat Jun 12 15:38:41 2021 +0900

    [SPARK-35746][UI] Fix taskid in the stage page task event timeline
    
    ### What changes were proposed in this pull request?
    Task id is given incorrect in the timeline plot in Stage Page
    
    ### Why are the changes needed?
    Map event timeline plots to correct task
    **Before:**
    
![image](https://user-images.githubusercontent.com/23054875/121761077-81775800-cb4b-11eb-8ec6-ee71926a6549.png)
    
    **After**
    
![image](https://user-images.githubusercontent.com/23054875/121761195-02ceea80-cb4c-11eb-8ce6-07bb1cca190e.png)
    ### Does this PR introduce _any_ user-facing change?
    
    No
    
    ### How was this patch tested?
    
    Manually tested
    
    Closes #32888 from shahidki31/shahid/fixtaskid.
    
    Authored-by: shahid <shahidk...@gmail.com>
    Signed-off-by: Kousuke Saruta <saru...@oss.nttdata.com>
    (cherry picked from commit 450b415028c3b00f3a002126cd11318d3932e28f)
    Signed-off-by: Kousuke Saruta <saru...@oss.nttdata.com>
---
 core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala 
b/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala
index ccaa70b..e9eb62e 100644
--- a/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala
+++ b/core/src/main/scala/org/apache/spark/ui/jobs/StagePage.scala
@@ -352,7 +352,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 " + index + " (attempt " + attempt + 
")"}<br>
+                 |data-title="${s"Task " + taskInfo.taskId + " (attempt " + 
attempt + ")"}<br>
                  |Status: ${taskInfo.status}<br>
                  |Launch Time: ${UIUtils.formatDate(new Date(launchTime))}
                  |${

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

Reply via email to