MINOR: Remove Hack in TestCluster Task

original-brownbear · original-brownbear · commit edf0b21cdde3 · 2018-11-13T11:33:45.000+01:00
* The logic that splits by `","` isn't necessary anymore since elastic#35375 removed the hack that set the comma separated list for the unicast host
diff --git a/buildSrc/src/main/groovy/org/elasticsearch/gradle/test/ClusterFormationTasks.groovy b/buildSrc/src/main/groovy/org/elasticsearch/gradle/test/ClusterFormationTasks.groovy
@@ -719,7 +719,7 @@ class ClusterFormationTasks {
                 unicastHosts.addAll(node.config.otherUnicastHostAddresses.call())
                 String unicastHost = node.config.unicastTransportUri(node, null, project.ant)
                 if (unicastHost != null) {
-                    unicastHosts.addAll(Arrays.asList(unicastHost.split(",")))
+                    unicastHosts.add(unicastHost)
                 }
             }
             String unicastHostsTxt = String.join("\n", unicastHosts)

Original file line number	Diff line number	Diff line change
`@@ -719,7 +719,7 @@ class ClusterFormationTasks {`
`719`	`719`	`unicastHosts.addAll(node.config.otherUnicastHostAddresses.call())`
`720`	`720`	`String unicastHost = node.config.unicastTransportUri(node, null, project.ant)`
`721`	`721`	`if (unicastHost != null) {`
`722`		`- unicastHosts.addAll(Arrays.asList(unicastHost.split(",")))`
	`722`	`+ unicastHosts.add(unicastHost)`
`723`	`723`	`}`
`724`	`724`	`}`
`725`	`725`	`String unicastHostsTxt = String.join("\n", unicastHosts)`