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 14e77ab [MINOR][UI] Correct JobPage's skipped/pending tableHeaderId
14e77ab is described below
commit 14e77abbf869753151cc9b3cf54ad0f58fc5e482
Author: linzebing <[email protected]>
AuthorDate: Sun Dec 13 22:00:05 2020 +0900
[MINOR][UI] Correct JobPage's skipped/pending tableHeaderId
### What changes were proposed in this pull request?
Current Spark Web UI job page's header link of pending/skipped stages is
inconsistent with their statuses. See the picture below:
### Why are the changes needed?
The code determining the `pendingOrSkippedTableId` has the wrong logic. As
explained in the code:
> If the job is completed, then any pending stages are displayed as
"skipped" [code
pointer](https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala#L266)
This PR fixes the logic for `pendingOrSkippedTableId` which aligns with the
stage statuses.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Verified that header link is consistent with stage status with the fix.
Closes #30749 from linzebing/ui_bug.
Authored-by: linzebing <[email protected]>
Signed-off-by: Kousuke Saruta <[email protected]>
(cherry picked from commit 0277fddaef17b615354c735a2c89cdced5f1d8f6)
Signed-off-by: Kousuke Saruta <[email protected]>
---
core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala
b/core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala
index 54582df..c80b6d1 100644
--- a/core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala
+++ b/core/src/main/scala/org/apache/spark/ui/jobs/JobPage.scala
@@ -280,9 +280,9 @@ private[ui] class JobPage(parent: JobsTab, store:
AppStatusStore) extends WebUIP
val pendingOrSkippedTableId =
if (isComplete) {
- "pending"
- } else {
"skipped"
+ } else {
+ "pending"
}
val activeStagesTable =
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
