Skip to content

[SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them … #8701

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.

Sign up for GitHub

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

Jump to bottom
Closed
tgravescs wants to merge 1 commit into from
Closed

[SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them … #8701

tgravescs wants to merge 1 commit into from

Conversation

@tgravescs
Copy link
Contributor

@tgravescs tgravescs commented Sep 10, 2015

Cherry-pick this to branch 1.5.

[SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them …
16e1c5f 
…are already running.

Author: Rohit Agarwal <rohita@qubole.com>

Closes apache#8153 from mindprince/SPARK-9924.
@vanzin
Copy link
Contributor

vanzin commented Sep 10, 2015

I didn't notice any changes from the original, is that right? LGTM either way.

@tgravescs
Copy link
Contributor Author

tgravescs commented Sep 10, 2015

yep, no changes just cherry-pick... I don't know that I needed the pr but figured would put it up anywa

@SparkQA
Copy link

SparkQA commented Sep 10, 2015

Test build #42274 has finished for PR 8701 at commit 16e1c5f.

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

@vanzin
Copy link
Contributor

vanzin commented Sep 10, 2015

failure is not related, but just in case. retest this please

@SparkQA
Copy link

SparkQA commented Sep 10, 2015

Test build #42291 has finished for PR 8701 at commit 16e1c5f.

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

@vanzin
Copy link
Contributor

vanzin commented Sep 10, 2015

wtf?

@vanzin
Copy link
Contributor

vanzin commented Sep 10, 2015

branch-1.5 should be fixed now. retest this please

@SparkQA
Copy link

SparkQA commented Sep 10, 2015

Test build #42300 has finished for PR 8701 at commit 16e1c5f.

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

@vanzin
Copy link
Contributor

vanzin commented Sep 10, 2015

Hmm seems like github wasn't synchronized yet. retest this please.

@SparkQA
Copy link

SparkQA commented Sep 11, 2015

Test build #42303 timed out for PR 8701 at commit 16e1c5f after a configured wait of 175m.

@tgravescs
Copy link
Contributor Author

tgravescs commented Sep 11, 2015

retest this please

@SparkQA
Copy link

SparkQA commented Sep 11, 2015

Test build #42337 has finished for PR 8701 at commit 16e1c5f.

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

@vanzin
Copy link
Contributor

vanzin commented Sep 11, 2015

merging

asfgit pushed a commit that referenced this pull request Sep 11, 2015
[SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them …
7f10bd6 
Cherry-pick this to branch 1.5.

Author: Rohit Agarwal <rohita@qubole.com>

Closes #8701 from tgravescs/SPARK-9924-1.5 and squashes the following commits:

16e1c5f [Rohit Agarwal] [SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them are already running.
@vanzin
Copy link
Contributor

vanzin commented Sep 11, 2015

@tgravescs could you close this? I don't think it's closed automatically since it's in a branch. Thanks!

@tgravescs
Copy link
Contributor Author

tgravescs commented Sep 11, 2015

thanks @vanzin, closing

@tgravescs tgravescs closed this Sep 11, 2015
ashangit pushed a commit to ashangit/spark that referenced this pull request Oct 19, 2016
[SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them …
a20c950 
Cherry-pick this to branch 1.5.

Author: Rohit Agarwal <rohita@qubole.com>

Closes apache#8701 from tgravescs/SPARK-9924-1.5 and squashes the following commits:

16e1c5f [Rohit Agarwal] [SPARK-9924] [WEB UI] Don't schedule checkForLogs while some of them are already running.

(cherry picked from commit 7f10bd6)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tgravescs @vanzin @SparkQA