prestodb / presto-yarn

57 stars 28 forks source link

tests: Wait before asserting that slider is stoppable #5

Closed anusudarsan closed 9 years ago

anusudarsan commented 9 years ago

The --wait argument in bin/slider stop is to delay the process of stopping the presto application, and not really waiting after the process has stopped. So just remove it and add a wait time in the tests (using retry )to confirm the process has stopped before we call the assertion statement.

SWARM-1405

kokosing commented 9 years ago

lgtm % give it a try ;)