Skip to content

Conversation

@Leemoonsoo
Copy link
Member

What is this PR for?

Fix flaky test ShellInterpreterTest#testShellTimeout https://issues.apache.org/jira/browse/ZEPPELIN-2007

this PR makes sure testShellTimeout() produce timeout error by increase gap between timeout and test value (from 61-60 = 1 to 4-2 = 2).

What type of PR is it?

Hot Fix

Todos

  • - Fix

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2007

Questions:

  • Does the licenses files need update? no
  • Is there breaking changes for older versions? no
  • Does this needs documentation? no

@Leemoonsoo
Copy link
Member Author

I'm merging this to master as a hotfix.

@asfgit asfgit closed this in 2ef8359 Jan 24, 2017
asfgit pushed a commit that referenced this pull request Jan 25, 2017
### What is this PR for?
Fix flaky test ShellInterpreterTest#testShellTimeout https://issues.apache.org/jira/browse/ZEPPELIN-2007

this PR makes sure `testShellTimeout()` produce timeout error by increase gap between timeout and test value (from `61-60 = 1`  to `4-2 = 2`).

### What type of PR is it?
Hot Fix

### Todos
* [x] - Fix

### What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2007

### Questions:
* Does the licenses files need update? no
* Is there breaking changes for older versions? no
* Does this needs documentation? no

Author: Lee moon soo <[email protected]>

Closes #1939 from Leemoonsoo/ZEPPELIN-2007 and squashes the following commits:

57c6a9b [Lee moon soo] Make ShellInterpreterTest faster, more reliable

(cherry picked from commit 2ef8359)
Signed-off-by: Mina Lee <[email protected]>
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.

1 participant