You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by mr...@apache.org on 2019/01/14 13:51:29 UTC

svn commit: r1851246 - /jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongodProcessFactory.java

Author: mreutegg
Date: Mon Jan 14 13:51:28 2019
New Revision: 1851246

URL: http://svn.apache.org/viewvc?rev=1851246&view=rev
Log:
OAK-7986: Incorrect MongoDB URI in ReplicaSetDefaultWriteConcernIT

Modified:
    jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongodProcessFactory.java

Modified: jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongodProcessFactory.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongodProcessFactory.java?rev=1851246&r1=1851245&r2=1851246&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongodProcessFactory.java (original)
+++ jackrabbit/oak/trunk/oak-store-document/src/test/java/org/apache/jackrabbit/oak/plugins/document/mongo/MongodProcessFactory.java Mon Jan 14 13:51:28 2019
@@ -135,12 +135,15 @@ public class MongodProcessFactory extend
     }
 
     static String localhost(Iterable<Integer> ports) {
-        String portsString = Joiner.on(',').join(ports);
         String host = InetAddress.getLoopbackAddress().getHostAddress();
-        if (!portsString.isEmpty()) {
-            host += ":" + portsString;
+        List<String> portsWithHost = new ArrayList<>();
+        for (int p : ports) {
+            portsWithHost.add(host + ":" + p);
         }
-        return host;
+        if (portsWithHost.isEmpty()) {
+            return host;
+        }
+        return Joiner.on(',').join(portsWithHost);
     }
 
     //----------------------------< internal >----------------------------------