Phillippko commented on code in PR #6721:
URL: https://github.com/apache/ignite-3/pull/6721#discussion_r2414027970
##########
modules/compatibility-tests/src/testFixtures/java/org/apache/ignite/internal/IgniteCluster.java:
##########
@@ -114,6 +120,15 @@ public void start(String igniteVersion, int nodesCount,
List<String> extraIgnite
* @param nodesCount Number of nodes in the cluster.
*/
public void startEmbedded(int nodesCount, boolean initCluster) {
+ startEmbedded(null, nodesCount, initCluster);
+ }
+
+ /**
+ * Starts cluster in embedded mode with nodes of current version.
+ *
+ * @param nodesCount Number of nodes in the cluster.
+ */
+ public void startEmbedded(@Nullable TestInfo testInfo, int nodesCount,
boolean initCluster) {
Review Comment:
Right now IgniteCluster uses **default** configuration for all nodes, i.e.
they all try to start on the port 3344. ```testInfo``` is not needed yet, but I
wanted to take code from Cluster#startEmbedded (in ignite-runner testFixtures
module) so there are no differences in behaviour
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
