(ignite-3) branch main updated: IGNITE-21576 Unmute tests in IndexManagerTest (#3363)

[apolovtsev]

This is an automated email from the ASF dual-hosted git repository.

apolovtsev 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 36885b92aa IGNITE-21576 Unmute tests in IndexManagerTest (#3363)
36885b92aa is described below

commit 36885b92aab614579b747c28237bab8fa5375be4
Author: Alexander Polovtcev <alex.polovt...@gmail.com>
AuthorDate: Thu Mar 7 17:26:15 2024 +0200

    IGNITE-21576 Unmute tests in IndexManagerTest (#3363)
---
 .../java/org/apache/ignite/internal/index/IndexManagerTest.java    | 7 ++-----
 .../src/main/java/org/apache/ignite/internal/table/TableImpl.java  | 3 +--
 2 files changed, 3 insertions(+), 7 deletions(-)

diff --git 
a/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java
 
b/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java
index 72e809b7d0..b6ee9dee73 100644
--- 
a/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java
+++ 
b/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java
@@ -78,7 +78,6 @@ import org.apache.ignite.internal.util.IgniteUtils;
 import org.apache.ignite.sql.IgniteSql;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 
@@ -136,12 +135,11 @@ public class IndexManagerTest extends 
BaseIgniteAbstractTest {
         verify(tableViewInternal, never()).unregisterIndex(anyInt());
     }
 
-    @Disabled("https://issues.apache.org/jira/browse/IGNITE-21576";)
     @Test
     void testDestroyIndex() throws Exception {
         createIndex(TABLE_NAME, INDEX_NAME);
 
-        CatalogIndexDescriptor indexDescriptor = 
catalogManager.aliveIndex(INDEX_NAME, catalogManager.latestCatalogVersion());
+        CatalogIndexDescriptor indexDescriptor = 
catalogManager.aliveIndex(INDEX_NAME, clock.nowLong());
         int indexId = indexDescriptor.id();
         int tableId = indexDescriptor.tableId();
 
@@ -154,12 +152,11 @@ public class IndexManagerTest extends 
BaseIgniteAbstractTest {
         verify(mvTableStorage).destroyIndex(indexId);
     }
 
-    @Disabled("https://issues.apache.org/jira/browse/IGNITE-21576";)
     @Test
     void testIndexDestroyedWithTable() throws Exception {
         createIndex(TABLE_NAME, INDEX_NAME);
 
-        CatalogIndexDescriptor indexDescriptor = 
catalogManager.aliveIndex(INDEX_NAME, catalogManager.latestCatalogVersion());
+        CatalogIndexDescriptor indexDescriptor = 
catalogManager.aliveIndex(INDEX_NAME, clock.nowLong());
         int indexId = indexDescriptor.id();
         int tableId = indexDescriptor.tableId();
 
diff --git 
a/modules/table/src/main/java/org/apache/ignite/internal/table/TableImpl.java 
b/modules/table/src/main/java/org/apache/ignite/internal/table/TableImpl.java
index 3452bfcb7c..e29c7667aa 100644
--- 
a/modules/table/src/main/java/org/apache/ignite/internal/table/TableImpl.java
+++ 
b/modules/table/src/main/java/org/apache/ignite/internal/table/TableImpl.java
@@ -289,7 +289,6 @@ public class TableImpl implements TableViewInternal {
     public void unregisterIndex(int indexId) {
         indexWrapperById.remove(indexId);
 
-        // TODO: https://issues.apache.org/jira/browse/IGNITE-21576 Also need 
to destroy the index storages
-        // tbl.storage().destroyIndex(indexId);
+        tbl.storage().destroyIndex(indexId);
     }
 }