Skip to content
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

[SPARK-9806] [WEB UI] Don't share ReplayListenerBus between multiple applications #8088

Closed
wants to merge 1 commit into from

Conversation

rohitagarwal003
Copy link
Contributor

No description provided.

@JoshRosen
Copy link
Contributor

Jenkins, this is ok to test.

@JoshRosen
Copy link
Contributor

/cc @vanzin

@JoshRosen
Copy link
Contributor

Jenkins, retest this please.

1 similar comment
@JoshRosen
Copy link
Contributor

Jenkins, retest this please.

@JoshRosen
Copy link
Contributor

Jenkins has been really flaky lately :(

@rohitagarwal003
Copy link
Contributor Author

Yeah, I noticed that in my other PR as well. :-(

@vanzin
Copy link
Contributor

vanzin commented Aug 11, 2015

LGTM. Ah, jenkins... retest this please.

@andrewor14
Copy link
Contributor

LGTM2

@SparkQA
Copy link

SparkQA commented Aug 11, 2015

Test build #40498 has finished for PR 8088 at commit 972af81.

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

@SparkQA
Copy link

SparkQA commented Aug 11, 2015

Test build #1455 has finished for PR 8088 at commit 972af81.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@andrewor14
Copy link
Contributor

retest this please

@vanzin
Copy link
Contributor

vanzin commented Aug 11, 2015

Wow, this is failing scala style tests on already checked in code... :-/

@vanzin
Copy link
Contributor

vanzin commented Aug 11, 2015

one last try. retest this please

@SparkQA
Copy link

SparkQA commented Aug 12, 2015

Test build #40535 timed out for PR 8088 at commit 972af81 after a configured wait of 175m.

@vanzin
Copy link
Contributor

vanzin commented Aug 12, 2015

sigh. retest this please

@SparkQA
Copy link

SparkQA commented Aug 12, 2015

Test build #40573 has finished for PR 8088 at commit 972af81.

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

@rohitagarwal003
Copy link
Contributor Author

Finally!

@andrewor14
Copy link
Contributor

Merging into master 1.5.

asfgit pushed a commit that referenced this pull request Aug 12, 2015
…applications

Author: Rohit Agarwal <rohita@qubole.com>

Closes #8088 from mindprince/SPARK-9806.

(cherry picked from commit a807fcb)
Signed-off-by: Andrew Or <andrew@databricks.com>
@asfgit asfgit closed this in a807fcb Aug 12, 2015
asfgit pushed a commit that referenced this pull request Aug 12, 2015
…applications

Author: Rohit Agarwal <rohita@qubole.com>

Closes #8088 from mindprince/SPARK-9806.

(cherry picked from commit a807fcb)
Signed-off-by: Andrew Or <andrew@databricks.com>
CodingCat pushed a commit to CodingCat/spark that referenced this pull request Aug 17, 2015
…applications

Author: Rohit Agarwal <rohita@qubole.com>

Closes apache#8088 from mindprince/SPARK-9806.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants