Skip to content

[FLINK-2831] [Storm Compatibility Tests] Fixed instable test - #1519

Closed
mjsax wants to merge 1 commit into
apache:masterfrom
mjsax:flink-2831-WrapperSetupHelperTest
Closed

[FLINK-2831] [Storm Compatibility Tests] Fixed instable test#1519
mjsax wants to merge 1 commit into
apache:masterfrom
mjsax:flink-2831-WrapperSetupHelperTest

Conversation

@mjsax

@mjsax mjsax commented Jan 18, 2016

Copy link
Copy Markdown
Member

No description provided.

@StephanEwen

Copy link
Copy Markdown
Contributor

Looks good. I think you can let the thread sleep a bit more, say 50 or 100 msecs. Fewer checks and context switched, probably still good enough test responsiveness.

@mjsax

mjsax commented Jan 19, 2016

Copy link
Copy Markdown
Member Author

Ok. Will increase sleep time and merge...

@asfgit asfgit closed this in 046f044 Jan 19, 2016
ChengXiangLi pushed a commit to ChengXiangLi/flink that referenced this pull request Jan 25, 2016
@mjsax
mjsax deleted the flink-2831-WrapperSetupHelperTest branch February 12, 2016 16:05
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.

3 participants