fix(retry): Fix retry strategy when all hosts are down (#356)

Author: ElPicador

.travis.yml

@@ -1,18 +1,20 @@
 language: java
 jdk:
-- oraclejdk7
+- openjdk7 #oraclejdk7 is not available on trusty because it's EOL
 - oraclejdk8
+
 after_success:
 - mvn -e -X clean test jacoco:report coveralls:jacoco
+
 env:
   global:
   - VERBOSE=1
 
-# force non-container build
-sudo: required
-# workaround for https://github.com/travis-ci/travis-ci/issues/5227
-before_install:
-  - cat /etc/hosts # optionally check the content *before*
-  - sudo hostname "$(hostname | cut -c1-63)"
-  - sed -e "s/^\\(127\\.0\\.0\\.1.*\\)/\\1 $(hostname | cut -c1-63)/" /etc/hosts | sudo tee /etc/hosts
-  - cat /etc/hosts # optionally check the content *after*
+## force non-container build
+#sudo: required
+## workaround for https://github.com/travis-ci/travis-ci/issues/5227
+#before_install:
+#  - cat /etc/hosts # optionally check the content *before*
+#  - sudo hostname "$(hostname | cut -c1-63)"
+#  - sed -e "s/^\\(127\\.0\\.0\\.1.*\\)/\\1 $(hostname | cut -c1-63)/" /etc/hosts | sudo tee /etc/hosts
+#  - cat /etc/hosts # optionally check the content *after*

ChangeLog

@@ -1,6 +1,10 @@
 Changelog
 =========
 
+1.12.1 (2017-10-09)
+-------------------
+- Fix retry strategy when all hosts are down
+
 1.12.0 (2017-08-22)
 -------------------
 - Rename Key to ApiKey

src/main/java/com/algolia/search/saas/APIClient.java

@@ -1157,7 +1157,12 @@ private List<String> hostsThatAreUp(List<String> hosts) {
         result.add(host);
       }
     }
-    return result;
+
+    if (result.isEmpty()) {
+      return hosts;
+    } else {
+      return result;
+    }
   }
 
   private boolean isHostUpOrCouldBeRetried(String host) {