This is an automated email from the ASF dual-hosted git repository.
mpochatkin pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/main by this push:
new c07e45c08dc IGNITE-25382 Remove compute thread pool size from test
configurations (#5812)
c07e45c08dc is described below
commit c07e45c08dce43929ebd05638f4aa1c3cadd6e39
Author: Vadim Pakhnushev <8614891+valep...@users.noreply.github.com>
AuthorDate: Fri May 16 13:09:09 2025 +0300
IGNITE-25382 Remove compute thread pool size from test configurations
(#5812)
---
.../ignite/internal/catalog/compaction/ItCatalogCompactionTest.java | 1 -
.../java/org/apache/ignite/internal/compute/ItComputeBaseTest.java | 2 ++
.../java/org/apache/ignite/internal/compute/ItComputeTestClient.java | 2 ++
.../configuration/validation/ConfigurationDuplicatesValidatorTest.java | 1 -
.../apache/ignite/internal/rest/compute/ItComputeControllerTest.java | 2 ++
.../org/apache/ignite/internal/client/ItClientDirectMappingTest.java | 1 -
.../java/org/apache/ignite/internal/ClusterPerClassIntegrationTest.java | 1 -
.../java/org/apache/ignite/internal/ClusterPerTestIntegrationTest.java | 1 -
.../apache/ignite/tx/distributed/ItTransactionPrimaryChangeTest.java | 1 -
.../java/org/apache/ignite/tx/distributed/ItTxResourcesVacuumTest.java | 1 -
10 files changed, 6 insertions(+), 7 deletions(-)
diff --git
a/modules/catalog-compaction/src/integrationTest/java/org/apache/ignite/internal/catalog/compaction/ItCatalogCompactionTest.java
b/modules/catalog-compaction/src/integrationTest/java/org/apache/ignite/internal/catalog/compaction/ItCatalogCompactionTest.java
index 20549ef564a..40048b23ce3 100644
---
a/modules/catalog-compaction/src/integrationTest/java/org/apache/ignite/internal/catalog/compaction/ItCatalogCompactionTest.java
+++
b/modules/catalog-compaction/src/integrationTest/java/org/apache/ignite/internal/catalog/compaction/ItCatalogCompactionTest.java
@@ -101,7 +101,6 @@ class ItCatalogCompactionTest extends
ClusterPerClassIntegrationTest {
+ " },\n"
+ " clientConnector.port: {},\n"
+ " rest.port: {},\n"
- + " compute.threadPoolSize: 1,\n"
+ " failureHandler.dumpThreadsOnFailure: false\n"
+ "}";
}
diff --git
a/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java
b/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java
index d97b7ded0d2..f75f63f1034 100644
---
a/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java
+++
b/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeBaseTest.java
@@ -79,6 +79,7 @@ import org.apache.ignite.compute.TaskDescriptor;
import org.apache.ignite.compute.task.TaskExecution;
import org.apache.ignite.deployment.DeploymentUnit;
import org.apache.ignite.internal.ClusterPerClassIntegrationTest;
+import org.apache.ignite.internal.ConfigOverride;
import org.apache.ignite.internal.table.partition.HashPartition;
import org.apache.ignite.internal.testframework.IgniteTestUtils;
import org.apache.ignite.lang.CancelHandle;
@@ -104,6 +105,7 @@ import org.junit.jupiter.params.provider.ValueSource;
* corresponding job class to the jobs source set. The integration tests
depend on this source set so the job class will be visible and it
* will be automatically compiled and packed into the
ignite-integration-test-jobs-1.0-SNAPSHOT.jar.
*/
+@ConfigOverride(name = "ignite.compute.threadPoolSize", value = "1")
public abstract class ItComputeBaseTest extends ClusterPerClassIntegrationTest
{
protected abstract List<DeploymentUnit> units();
diff --git
a/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeTestClient.java
b/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeTestClient.java
index 83b8a25bb4b..199539ac4b1 100644
---
a/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeTestClient.java
+++
b/modules/compute/src/integrationTest/java/org/apache/ignite/internal/compute/ItComputeTestClient.java
@@ -21,6 +21,7 @@ import static
org.apache.ignite.internal.TestWrappers.unwrapIgniteImpl;
import org.apache.ignite.client.IgniteClient;
import org.apache.ignite.compute.IgniteCompute;
+import org.apache.ignite.internal.ConfigOverride;
import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeEach;
@@ -28,6 +29,7 @@ import org.junit.jupiter.api.BeforeEach;
* Integration tests for Compute functionality using thin client API.
*/
@SuppressWarnings("NewClassNamingConvention")
+@ConfigOverride(name = "ignite.compute.threadPoolSize", value = "1")
public class ItComputeTestClient extends ItComputeTestEmbedded {
private IgniteClient client;
diff --git
a/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/validation/ConfigurationDuplicatesValidatorTest.java
b/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/validation/ConfigurationDuplicatesValidatorTest.java
index 42846a64c9c..05c6c3891ed 100644
---
a/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/validation/ConfigurationDuplicatesValidatorTest.java
+++
b/modules/configuration/src/test/java/org/apache/ignite/internal/configuration/validation/ConfigurationDuplicatesValidatorTest.java
@@ -82,7 +82,6 @@ class ConfigurationDuplicatesValidatorTest extends
BaseIgniteAbstractTest {
+ " rest {\n"
+ " port: 123\n"
+ " },\n"
- + " compute.threadPoolSize: 1,\n"
+ " failureHandler.dumpThreadsOnFailure: false\n"
+ "}";
diff --git
a/modules/rest/src/integrationTest/java/org/apache/ignite/internal/rest/compute/ItComputeControllerTest.java
b/modules/rest/src/integrationTest/java/org/apache/ignite/internal/rest/compute/ItComputeControllerTest.java
index 2f83033185b..61870595dd0 100644
---
a/modules/rest/src/integrationTest/java/org/apache/ignite/internal/rest/compute/ItComputeControllerTest.java
+++
b/modules/rest/src/integrationTest/java/org/apache/ignite/internal/rest/compute/ItComputeControllerTest.java
@@ -58,6 +58,7 @@ import org.apache.ignite.compute.JobExecution;
import org.apache.ignite.compute.JobExecutionContext;
import org.apache.ignite.compute.JobTarget;
import org.apache.ignite.internal.ClusterPerClassIntegrationTest;
+import org.apache.ignite.internal.ConfigOverride;
import org.apache.ignite.internal.compute.ComputeComponentImpl;
import org.apache.ignite.internal.compute.ExecutionManager;
import org.apache.ignite.internal.compute.IgniteComputeImpl;
@@ -71,6 +72,7 @@ import org.junit.jupiter.api.Test;
* Integration tests for {@link ComputeController}.
*/
@MicronautTest
+@ConfigOverride(name = "ignite.compute.threadPoolSize", value = "1")
public class ItComputeControllerTest extends ClusterPerClassIntegrationTest {
private static final String COMPUTE_URL = "/management/v1/compute/";
diff --git
a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/client/ItClientDirectMappingTest.java
b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/client/ItClientDirectMappingTest.java
index 192869ec261..d54f6eac45e 100644
---
a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/client/ItClientDirectMappingTest.java
+++
b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/client/ItClientDirectMappingTest.java
@@ -54,7 +54,6 @@ public class ItClientDirectMappingTest extends
ClusterPerTestIntegrationTest {
+ " clientConnector: { port:{} },\n"
+ " rest.port: {},\n"
+ " raft: { responseTimeoutMillis: 30000 },"
- + " compute.threadPoolSize: 1,\n"
+ " failureHandler.dumpThreadsOnFailure: false\n"
+ "}";
diff --git
a/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerClassIntegrationTest.java
b/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerClassIntegrationTest.java
index ca35ca44324..d440aa84240 100644
---
a/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerClassIntegrationTest.java
+++
b/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerClassIntegrationTest.java
@@ -99,7 +99,6 @@ public abstract class ClusterPerClassIntegrationTest extends
BaseIgniteAbstractT
+ " clientConnector.port: {},\n"
+ " clientConnector.sendServerExceptionStackTraceToClient:
true,\n"
+ " rest.port: {},\n"
- + " compute.threadPoolSize: 1,\n"
+ " failureHandler.dumpThreadsOnFailure: false\n"
+ "}";
diff --git
a/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerTestIntegrationTest.java
b/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerTestIntegrationTest.java
index 5e235d3e554..4a6de379abc 100644
---
a/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerTestIntegrationTest.java
+++
b/modules/runner/src/testFixtures/java/org/apache/ignite/internal/ClusterPerTestIntegrationTest.java
@@ -70,7 +70,6 @@ public abstract class ClusterPerTestIntegrationTest extends
BaseIgniteAbstractTe
+ " },\n"
+ " clientConnector.port: {},\n"
+ " rest.port: {},\n"
- + " compute.threadPoolSize: 1,\n"
+ " failureHandler.dumpThreadsOnFailure: false\n"
+ "}";
diff --git
a/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTransactionPrimaryChangeTest.java
b/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTransactionPrimaryChangeTest.java
index 0a3faaec16a..4b23ed15d6c 100644
---
a/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTransactionPrimaryChangeTest.java
+++
b/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTransactionPrimaryChangeTest.java
@@ -66,7 +66,6 @@ public class ItTransactionPrimaryChangeTest extends
ClusterPerTestIntegrationTes
+ " clientConnector: { port:{} },\n"
+ " rest.port: {},\n"
+ " raft: { responseTimeoutMillis: 30000 },"
- + " compute.threadPoolSize: 1,\n"
+ " failureHandler.dumpThreadsOnFailure: false\n"
+ "}";
diff --git
a/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTxResourcesVacuumTest.java
b/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTxResourcesVacuumTest.java
index 60d7b312ef0..c653de79f56 100644
---
a/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTxResourcesVacuumTest.java
+++
b/modules/transactions/src/integrationTest/java/org/apache/ignite/tx/distributed/ItTxResourcesVacuumTest.java
@@ -116,7 +116,6 @@ public class ItTxResourcesVacuumTest extends
ClusterPerTestIntegrationTest {
+ " clientConnector: { port:{} },\n"
+ " rest.port: {},\n"
+ " raft: { responseTimeoutMillis: 30000 },"
- + " compute.threadPoolSize: 1,\n"
+ " failureHandler.dumpThreadsOnFailure: false\n"
+ "}";
