Skip to content

[SPARK-24704][WebUI] Fix the order of stages in the DAG graph#21680

Closed
stanzhai wants to merge 1 commit intoapache:masterfrom
stanzhai:fix-dag-graph
Closed

[SPARK-24704][WebUI] Fix the order of stages in the DAG graph#21680
stanzhai wants to merge 1 commit intoapache:masterfrom
stanzhai:fix-dag-graph

Conversation

@stanzhai
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Before:

wx20180630-155537

After:

wx20180630-155604

How was this patch tested?

Manual tests.

Copy link
Copy Markdown
Member

@felixcheung felixcheung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok looks reasonable?

@felixcheung
Copy link
Copy Markdown
Member

ok to test

@SparkQA
Copy link
Copy Markdown

SparkQA commented Jun 30, 2018

Test build #92501 has finished for PR 21680 at commit b3420d6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

Copy link
Copy Markdown
Contributor

@jiangxb1987 jiangxb1987 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hvanhovell
Copy link
Copy Markdown
Contributor

Merging to master. Thanks!

@asfgit asfgit closed this in 772060d Jul 4, 2018
asfgit pushed a commit that referenced this pull request Aug 28, 2018
## What changes were proposed in this pull request?

Before:

![wx20180630-155537](https://user-images.githubusercontent.com/1438757/42123357-2c2e2d84-7c83-11e8-8abd-1c2860f38783.png)

After:

![wx20180630-155604](https://user-images.githubusercontent.com/1438757/42123359-32fae990-7c83-11e8-8a7b-cdcee94f9123.png)

## How was this patch tested?

Manual tests.

Author: Stan Zhai <mail@stanzhai.site>

Closes #21680 from stanzhai/fix-dag-graph.

(cherry picked from commit 772060d)
Signed-off-by: Marcelo Vanzin <vanzin@cloudera.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants