Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

replace WaitUtil.sleep with Futures #518

Closed
jgangemi opened this issue Jul 20, 2016 · 0 comments
Closed

replace WaitUtil.sleep with Futures #518

jgangemi opened this issue Jul 20, 2016 · 0 comments
Assignees
@jgangemi
Labels
feature
Milestone
0.15.14

Comments

@jgangemi
Copy link
Collaborator

right now if you specify a wait time, the plugin just basically sleeps which means if the container exits sooner you still have to wait for the sleep to complete.

what (hopefully) would be better would be to do is use a Future what would allow the process to continue as soon as it completes and timeout if it takes longer then the allotted wait time.

thoughts?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jgangemi jgangemi

Labels
feature
Projects
None yet
Milestone
0.15.14
Development

Successfully merging a pull request may close this issue.

- use Futures instead of sleeping, request for feedback jgangemi/docker-maven-plugin
1 participant
@jgangemi