[FLINK-6567] ExecutionGraphMetricsTest fails on Windows CI

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 1.3.0, 1.4.0
• Fix Version/s: 1.5.0
• Component/s: Tests
• Labels: test-stability
• Assignee: Till Rohrmann
• Reporter: Chesnay Schepler
• Created: 12/May/17 12:43
• Updated: 03/Apr/18 14:37
• Resolved: 03/Apr/18 14:37

Description

The testExecutionGraphRestartTimeMetric fails every time i run it on AppVeyor. It also very rarely failed for me locally.

The test fails at Line 235 if the RUNNING timestamp is equal to the RESTARTING timestamp, which may happen when combining a fast test with a low resolution clock.

A simple fix would be to increase the timestamp between RUNNING and RESTARTING by adding a 50ms sleep timeout into the TestingRestartStrategy#canRestart() method, as this one is called before transitioning to the RESTARTING state.

Issue Links

• links to: GitHub Pull Request #5782