This is an automated email from the ASF dual-hosted git repository.
sk0x50 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 0e39e8e19d1 IGNITE-27026 Fix outdated TODOs (#7184)
0e39e8e19d1 is described below
commit 0e39e8e19d12d02abc06b95400ebb6118158dc7e
Author: Slava Koptilin <[email protected]>
AuthorDate: Tue Dec 9 13:39:34 2025 +0200
IGNITE-27026 Fix outdated TODOs (#7184)
---
.../java/org/apache/ignite/internal/replicator/TablePartitionId.java | 2 --
.../apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java | 2 +-
.../disaster/ItHighAvailablePartitionSequentialRecoveriesTest.java | 2 --
.../org/apache/ignite/internal/table/distributed/TableManager.java | 5 +++--
4 files changed, 4 insertions(+), 7 deletions(-)
diff --git
a/modules/core/src/main/java/org/apache/ignite/internal/replicator/TablePartitionId.java
b/modules/core/src/main/java/org/apache/ignite/internal/replicator/TablePartitionId.java
index b9eca736b0e..f275f34b455 100644
---
a/modules/core/src/main/java/org/apache/ignite/internal/replicator/TablePartitionId.java
+++
b/modules/core/src/main/java/org/apache/ignite/internal/replicator/TablePartitionId.java
@@ -17,8 +17,6 @@
package org.apache.ignite.internal.replicator;
-// TODO: https://issues.apache.org/jira/browse/IGNITE-19170 Should be
refactored to ZonePartitionId.
-
import java.util.regex.Pattern;
/**
diff --git
a/modules/distribution-zones/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java
b/modules/distribution-zones/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java
index 0a24da31e0c..845b7cbec7f 100644
---
a/modules/distribution-zones/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java
+++
b/modules/distribution-zones/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java
@@ -662,7 +662,7 @@ public class ItRebalanceDistributedTest extends
BaseIgniteAbstractTest {
@Test
@UseTestTxStateStorage
@UseRocksMetaStorage
- @Disabled("https://issues.apache.org/jira/browse/IGNITE-19170")
+ @Disabled("https://issues.apache.org/jira/browse/IGNITE-24345")
void testDestroyPartitionStoragesOnRestartEvictedNode(TestInfo testInfo)
throws Exception {
Node node = getNode(0);
diff --git
a/modules/table/src/integrationTest/java/org/apache/ignite/internal/table/distributed/disaster/ItHighAvailablePartitionSequentialRecoveriesTest.java
b/modules/table/src/integrationTest/java/org/apache/ignite/internal/table/distributed/disaster/ItHighAvailablePartitionSequentialRecoveriesTest.java
index e03fc001a23..6391fe5bb27 100644
---
a/modules/table/src/integrationTest/java/org/apache/ignite/internal/table/distributed/disaster/ItHighAvailablePartitionSequentialRecoveriesTest.java
+++
b/modules/table/src/integrationTest/java/org/apache/ignite/internal/table/distributed/disaster/ItHighAvailablePartitionSequentialRecoveriesTest.java
@@ -25,7 +25,6 @@ import static org.hamcrest.Matchers.is;
import java.util.List;
import org.apache.ignite.internal.app.IgniteImpl;
import org.apache.ignite.table.Table;
-import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test;
/** Test multiple HA zone partitions recovery in a row. */
@@ -41,7 +40,6 @@ public class ItHighAvailablePartitionSequentialRecoveriesTest
extends AbstractHi
}
@Test
- @Disabled("https://issues.apache.org/jira/browse/IGNITE-24061")
void testTwoSequentialResets() throws InterruptedException {
createHaZoneWithTable();
diff --git
a/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java
b/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java
index 0ced11ce45d..75b718d2ae2 100644
---
a/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java
+++
b/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java
@@ -1598,8 +1598,9 @@ public class TableManager implements
IgniteTablesInternal, IgniteComponent {
return writeLockAcquisitionFuture.thenCompose(stamp -> {
CompletableFuture<?>[] stopReplicaAndDestroyFutures = new
CompletableFuture<?>[partitions];
- // TODO https://issues.apache.org/jira/browse/IGNITE-19170
Partitions should be stopped on the assignments change
- // event triggered by zone drop or alter. Stop replica
asynchronously, out of metastorage event pipeline.
+ // TODO https://issues.apache.org/jira/browse/IGNITE-24345
+ // Partitions should be stopped on the assignments change
event triggered by zone drop or alter.
+ // Stop replica asynchronously, out of metastorage event
pipeline.
for (int partitionId = 0; partitionId < partitions;
partitionId++) {
CompletableFuture<Void> resourcesUnloadFuture;