Skip to content

[FLINK-2915] Fix for JobManagerProcessFailureBatchRecoveryITCase Test#2027

Closed
rekhajoshm wants to merge 1 commit intoapache:masterfrom
rekhajoshm:FLINK-2915
Closed

[FLINK-2915] Fix for JobManagerProcessFailureBatchRecoveryITCase Test#2027
rekhajoshm wants to merge 1 commit intoapache:masterfrom
rekhajoshm:FLINK-2915

Conversation

@rekhajoshm
Copy link
Copy Markdown
Contributor

[FLINK-2915] Fix for JobManagerProcessFailureBatchRecoveryITCase Test

@StephanEwen
Copy link
Copy Markdown
Contributor

Thanks for looking into this!
I think that increasing the test timeout will not solve the problem. In fact, the CI system kills stalled tests after 5 minutes anyways, that would be even before the 10 minutes timeout is met.

Looking at the commit history, I think that the test was already improved a bit since the issue was opened. The problem may actually have been fixed already.

@StephanEwen
Copy link
Copy Markdown
Contributor

I would vote to close the issue and pull request for now, and see if the test is stable now.

@rekhajoshm
Copy link
Copy Markdown
Contributor Author

Thanks @StephanEwen for review. Closing PR.

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