Skip to content

Conversation

@zsxwing
Copy link
Member

@zsxwing zsxwing commented Dec 13, 2016

What changes were proposed in this pull request?

This PR fixes the timeout value in awaitResultInForkJoinSafely for 2.1 and 2.0. Master has been fixed by #16230.

How was this patch tested?

Jenkins

@srowen
Copy link
Member

srowen commented Dec 13, 2016

I see, so there is no point in applying this to master, because it's as if it were already superseded by a bigger fix? but we 'back port' this stop-gap to earlier branches.

@zsxwing
Copy link
Member Author

zsxwing commented Dec 13, 2016

Yep. I want to backport this obvious fix without the entire patch for master.

@SparkQA
Copy link

SparkQA commented Dec 13, 2016

Test build #70091 has finished for PR 16268 at commit a664cd5.

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

@yhuai
Copy link
Contributor

yhuai commented Dec 13, 2016

LGTM

@zsxwing
Copy link
Member Author

zsxwing commented Dec 13, 2016

Thanks! Merging to 2.1 and 2.0.

asfgit pushed a commit that referenced this pull request Dec 13, 2016
…h 2.1, 2.0)

## What changes were proposed in this pull request?

This PR fixes the timeout value in `awaitResultInForkJoinSafely` for 2.1 and 2.0. Master has been fixed by #16230.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #16268 from zsxwing/SPARK-18843.
asfgit pushed a commit that referenced this pull request Dec 13, 2016
…h 2.1, 2.0)

## What changes were proposed in this pull request?

This PR fixes the timeout value in `awaitResultInForkJoinSafely` for 2.1 and 2.0. Master has been fixed by #16230.

## How was this patch tested?

Jenkins

Author: Shixiong Zhu <[email protected]>

Closes #16268 from zsxwing/SPARK-18843.

(cherry picked from commit f672bfd)
Signed-off-by: Shixiong Zhu <[email protected]>
@zsxwing zsxwing closed this Dec 13, 2016
@zsxwing zsxwing deleted the SPARK-18843 branch December 13, 2016 22:41
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.

4 participants