Skip to content

Conversation

@carsonwang
Copy link
Contributor

To reproduce the issue, go to the stage page and click DAG Visualization once, then go to the job page to show the job DAG visualization. You will only see the first stage of the job.
Root cause: the java script use local storage to remember your selection. Once you click the stage DAG visualization, the local storage set expand-dag-viz-arrow-stage to true. When you go to the job page, the js checks expand-dag-viz-arrow-stage in the local storage first and will try to show stage DAG visualization on the job page.
To fix this, I set an id to the DAG span to differ job page and stage page. In the js code, we check the id and local storage together to make sure we show the correct DAG visualization.

@carsonwang
Copy link
Contributor Author

retest this please

@JoshRosen
Copy link
Contributor

Jenkins, retest this please.

@JoshRosen
Copy link
Contributor

/cc @andrewor14 for review.

@andrewor14
Copy link
Contributor

cc @zsxwing

@andrewor14
Copy link
Contributor

Actually, this LGTM. Thanks for fixing another blocker! I'm going to try this out locally.

@andrewor14
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Aug 11, 2015

Test build #40465 timed out for PR 8104 at commit f3119a4 after a configured wait of 175m.

@SparkQA
Copy link

SparkQA commented Aug 11, 2015

Test build #40477 has finished for PR 8104 at commit f3119a4.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@andrewor14
Copy link
Contributor

retest this please

2 similar comments
@andrewor14
Copy link
Contributor

retest this please

@andrewor14
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Aug 12, 2015

Test build #40515 has finished for PR 8104 at commit f3119a4.

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

@SparkQA
Copy link

SparkQA commented Aug 12, 2015

Test build #1457 has finished for PR 8104 at commit f3119a4.

  • This patch fails Spark unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA
Copy link

SparkQA commented Aug 12, 2015

Test build #40520 timed out for PR 8104 at commit f3119a4 after a configured wait of 175m.

@zsxwing
Copy link
Member

zsxwing commented Aug 12, 2015

LGTM

@andrewor14
Copy link
Contributor

This actually passed tests. I'm merging into master 1.5.

asfgit pushed a commit that referenced this pull request Aug 12, 2015
To reproduce the issue, go to the stage page and click DAG Visualization once, then go to the job page to show the job DAG visualization. You will only see the first stage of the job.
Root cause: the java script use local storage to remember your selection. Once you click the stage DAG visualization, the local storage set `expand-dag-viz-arrow-stage` to true. When you go to the job page, the js checks `expand-dag-viz-arrow-stage` in the local storage first and will try to show stage DAG visualization on the job page.
To fix this, I set an id to the DAG span to differ job page and stage page. In the js code, we check the id and local storage together to make sure we show the correct DAG visualization.

Author: Carson Wang <carson.wang@intel.com>

Closes #8104 from carsonwang/SPARK-9426.

(cherry picked from commit bab8923)
Signed-off-by: Andrew Or <andrew@databricks.com>
@asfgit asfgit closed this in bab8923 Aug 12, 2015
@carsonwang carsonwang deleted the SPARK-9426 branch August 17, 2015 01:22
CodingCat pushed a commit to CodingCat/spark that referenced this pull request Aug 17, 2015
To reproduce the issue, go to the stage page and click DAG Visualization once, then go to the job page to show the job DAG visualization. You will only see the first stage of the job.
Root cause: the java script use local storage to remember your selection. Once you click the stage DAG visualization, the local storage set `expand-dag-viz-arrow-stage` to true. When you go to the job page, the js checks `expand-dag-viz-arrow-stage` in the local storage first and will try to show stage DAG visualization on the job page.
To fix this, I set an id to the DAG span to differ job page and stage page. In the js code, we check the id and local storage together to make sure we show the correct DAG visualization.

Author: Carson Wang <carson.wang@intel.com>

Closes apache#8104 from carsonwang/SPARK-9426.
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