[FLINK-9674] Remove 65s sleep in QueryableState E2E test

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.6.0
• Fix Version/s: 1.6.0
• Component/s: Runtime / Queryable State, Tests
• Labels: pull-request-available
• Assignee: Chesnay Schepler
• Reporter: Chesnay Schepler
• Created: 27/Jun/18 09:33
• Updated: 03/Jul/18 10:02
• Resolved: 03/Jul/18 10:00

Description

The test_queryable_state_restart_tm.sh kills a taskmanager, waits for the loss to be noticed, starts a new tm and waits for the job to continue.

kill_random_taskmanager

[...]

sleep 65 # this is a little longer than the heartbeat timeout so that the TM is gone

start_and_wait_for_tm

Instead of waiting for a fixed amount of time that is tied to some config value we should wait for a specific event, like the job being canceled.

Issue Links

• links to: GitHub Pull Request #6216