Skip to content

[SPARK-7350][Streaming][WebUI] Attach the Streaming tab when calling ssc.start() #5898

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

zsxwing
Copy link
Member

@zsxwing zsxwing commented May 5, 2015

It's meaningless to display the Streaming tab before ssc.start(). So we should attach it in the ssc.start method.

It's meaningless to display the Streaming tab before `ssc.start()`. So we should attach it in the `ssc.start` method.
@SparkQA
Copy link

SparkQA commented May 5, 2015

Test build #31802 has finished for PR 5898 at commit e676487.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@zsxwing
Copy link
Member Author

zsxwing commented May 5, 2015

rest this please.

@zsxwing
Copy link
Member Author

zsxwing commented May 5, 2015

retest this please.

@SparkQA
Copy link

SparkQA commented May 5, 2015

Test build #31810 has finished for PR 5898 at commit e676487.

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

@zsxwing
Copy link
Member Author

zsxwing commented May 5, 2015

retest this please.

@SparkQA
Copy link

SparkQA commented May 5, 2015

Test build #31825 has finished for PR 5898 at commit e676487.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@srowen
Copy link
Member

srowen commented May 5, 2015

The test failure is unrelated. This makes some sense to me. @tdas?

@tdas
Copy link
Contributor

tdas commented May 5, 2015

retest this please

@tdas
Copy link
Contributor

tdas commented May 5, 2015

LGTM. Will merge when tests pass.

@SparkQA
Copy link

SparkQA commented May 5, 2015

Test build #31852 has finished for PR 5898 at commit e676487.

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

@tdas
Copy link
Contributor

tdas commented May 5, 2015

retest this please

@MechCoder
Copy link
Contributor

I'm getting unrelated test failures too..

@SparkQA
Copy link

SparkQA commented May 5, 2015

Test build #31867 has finished for PR 5898 at commit e676487.

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

@asfgit asfgit closed this in c6d1efb May 5, 2015
asfgit pushed a commit that referenced this pull request May 5, 2015
…g ssc.start()

It's meaningless to display the Streaming tab before `ssc.start()`. So we should attach it in the `ssc.start` method.

Author: zsxwing <zsxwing@gmail.com>

Closes #5898 from zsxwing/SPARK-7350 and squashes the following commits:

e676487 [zsxwing] Attach the Streaming tab when calling ssc.start()

(cherry picked from commit c6d1efb)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@zsxwing zsxwing deleted the SPARK-7350 branch May 5, 2015 14:13
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
…g ssc.start()

It's meaningless to display the Streaming tab before `ssc.start()`. So we should attach it in the `ssc.start` method.

Author: zsxwing <zsxwing@gmail.com>

Closes apache#5898 from zsxwing/SPARK-7350 and squashes the following commits:

e676487 [zsxwing] Attach the Streaming tab when calling ssc.start()
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
…g ssc.start()

It's meaningless to display the Streaming tab before `ssc.start()`. So we should attach it in the `ssc.start` method.

Author: zsxwing <zsxwing@gmail.com>

Closes apache#5898 from zsxwing/SPARK-7350 and squashes the following commits:

e676487 [zsxwing] Attach the Streaming tab when calling ssc.start()
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
…g ssc.start()

It's meaningless to display the Streaming tab before `ssc.start()`. So we should attach it in the `ssc.start` method.

Author: zsxwing <zsxwing@gmail.com>

Closes apache#5898 from zsxwing/SPARK-7350 and squashes the following commits:

e676487 [zsxwing] Attach the Streaming tab when calling ssc.start()
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