Skip to content

Conversation

@iGabon
Copy link
Contributor

@iGabon iGabon commented Sep 9, 2016

Bugfix:donot throw RuntimeException when a error occured in awaitCompletion(long,TimeUnit)


This change is Reviewable

public boolean awaitCompletion(long timeout, TimeUnit timeUnit) throws InterruptedException {
return completed.await(timeout, timeUnit);
boolean result = completed.await(timeout, timeUnit);
getFirstError();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please fix indentation

@KostyaSha KostyaSha added this to the 3.0.6 milestone Sep 9, 2016
@KostyaSha KostyaSha merged commit d07e2f6 into docker-java:master Sep 9, 2016
tejksat pushed a commit to tejksat/docker-java that referenced this pull request Sep 13, 2016
docker-java#700)

* Bugfix:donot throw RuntimeException when a error occured in awaitCompletion(long,TimeUnit)

* Fix indentation

* fix format error,use docker-java-formatter.xml
panuse pushed a commit to TuKangTech/docker-java that referenced this pull request Aug 20, 2017
docker-java#700)

* Bugfix:donot throw RuntimeException when a error occured in awaitCompletion(long,TimeUnit)

* Fix indentation

* fix format error,use docker-java-formatter.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants