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

[BUILD][INFRA] bump the timeout to match the jenkins PRB #28666

Closed

Conversation

shaneknapp
Copy link
Contributor

What changes were proposed in this pull request?

bump the timeout to match what's set in jenkins

Why are the changes needed?

tests be timing out!

Does this PR introduce any user-facing change?

no

How was this patch tested?

via jenkins

@shaneknapp shaneknapp changed the title bump the timeout to match the jenkins PRB [BUILD][INFRA] bump the timeout to match the jenkins PRB May 28, 2020
@shaneknapp
Copy link
Contributor Author

i'm going to go ahead and merge this now, as it's blocking maven PRs.

@srowen @HyukjinKwon @sarutak

@SparkQA
Copy link

SparkQA commented May 28, 2020

Test build #123243 has finished for PR 28666 at commit 3a4b29c.

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

@shaneknapp
Copy link
Contributor Author

also going to backport this to 3.0 and 2.4

shaneknapp added a commit that referenced this pull request May 28, 2020
### What changes were proposed in this pull request?

bump the timeout to match what's set in jenkins

### Why are the changes needed?

tests be timing out!

### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?

via jenkins

Closes #28666 from shaneknapp/increase-jenkins-timeout.

Authored-by: shane knapp <incomplete@gmail.com>
Signed-off-by: shane knapp <incomplete@gmail.com>
(cherry picked from commit 9e68aff)
Signed-off-by: shane knapp <incomplete@gmail.com>
shaneknapp added a commit that referenced this pull request May 28, 2020
### What changes were proposed in this pull request?

bump the timeout to match what's set in jenkins

### Why are the changes needed?

tests be timing out!

### Does this PR introduce _any_ user-facing change?

no

### How was this patch tested?

via jenkins

Closes #28666 from shaneknapp/increase-jenkins-timeout.

Authored-by: shane knapp <incomplete@gmail.com>
Signed-off-by: shane knapp <incomplete@gmail.com>
(cherry picked from commit 9e68aff)
Signed-off-by: shane knapp <incomplete@gmail.com>
@shaneknapp
Copy link
Contributor Author

done!

@shaneknapp shaneknapp deleted the increase-jenkins-timeout branch May 28, 2020 21:29
@sarutak
Copy link
Member

sarutak commented May 28, 2020

@shaneknapp Thanks!

@dongjoon-hyun
Copy link
Member

+1, late LGTM. Thank you, @shaneknapp .

@HyukjinKwon
Copy link
Member

+1 LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants