This is an automated email from the ASF dual-hosted git repository. yuqi4733 pushed a commit to branch issue_3968 in repository https://gitbox.apache.org/repos/asf/gravitino.git
commit 8ceca55dfa28d86bd6285d281212271c38df7597 Author: yuqi <[email protected]> AuthorDate: Thu Jul 4 15:00:27 2024 +0800 fix --- .../gravitino/catalog/hadoop/TestHadoopCatalogOperations.java | 1 - .../gravitino/catalog/lakehouse/paimon/TestGravitinoPaimonTable.java | 2 -- core/src/main/java/com/datastrato/gravitino/EntityStoreFactory.java | 2 +- .../java/com/datastrato/gravitino/storage/kv/TestEntityKeyEncoding.java | 2 +- .../java/com/datastrato/gravitino/storage/kv/TestKvEntityStorage.java | 2 +- .../com/datastrato/gravitino/storage/kv/TestKvGarbageCollector.java | 2 +- .../com/datastrato/gravitino/storage/kv/TestKvNameMappingService.java | 2 +- .../java/com/datastrato/gravitino/storage/kv/TestRocksDBKvBackend.java | 2 ++ .../java/com/datastrato/gravitino/storage/kv/TestStorageVersion.java | 2 +- .../com/datastrato/gravitino/storage/kv/TestTransactionIdGenerator.java | 1 + .../com/datastrato/gravitino/storage/kv/TestTransactionalKvBackend.java | 1 + gradle.properties | 2 +- 12 files changed, 11 insertions(+), 10 deletions(-) diff --git a/catalogs/catalog-hadoop/src/test/java/com/datastrato/gravitino/catalog/hadoop/TestHadoopCatalogOperations.java b/catalogs/catalog-hadoop/src/test/java/com/datastrato/gravitino/catalog/hadoop/TestHadoopCatalogOperations.java index 407ccc6a5..7023ad96e 100644 --- a/catalogs/catalog-hadoop/src/test/java/com/datastrato/gravitino/catalog/hadoop/TestHadoopCatalogOperations.java +++ b/catalogs/catalog-hadoop/src/test/java/com/datastrato/gravitino/catalog/hadoop/TestHadoopCatalogOperations.java @@ -188,7 +188,6 @@ public class TestHadoopCatalogOperations { .delete(new Path(TEST_ROOT_PATH), true); File f = FileUtils.getFile(H2_file); - System.gc(); f.delete(); } diff --git a/catalogs/catalog-lakehouse-paimon/src/test/java/com/datastrato/gravitino/catalog/lakehouse/paimon/TestGravitinoPaimonTable.java b/catalogs/catalog-lakehouse-paimon/src/test/java/com/datastrato/gravitino/catalog/lakehouse/paimon/TestGravitinoPaimonTable.java index 32f1cdbdb..1c8b40fe6 100644 --- a/catalogs/catalog-lakehouse-paimon/src/test/java/com/datastrato/gravitino/catalog/lakehouse/paimon/TestGravitinoPaimonTable.java +++ b/catalogs/catalog-lakehouse-paimon/src/test/java/com/datastrato/gravitino/catalog/lakehouse/paimon/TestGravitinoPaimonTable.java @@ -108,8 +108,6 @@ public class TestGravitinoPaimonTable { Files.delete(Paths.get(warehousePath)); } catch (Exception e) { // Ignore - - System.out.println(e); } } diff --git a/core/src/main/java/com/datastrato/gravitino/EntityStoreFactory.java b/core/src/main/java/com/datastrato/gravitino/EntityStoreFactory.java index f0f5e3565..bf9b47276 100644 --- a/core/src/main/java/com/datastrato/gravitino/EntityStoreFactory.java +++ b/core/src/main/java/com/datastrato/gravitino/EntityStoreFactory.java @@ -58,7 +58,7 @@ public class EntityStoreFactory { if (KV_STORE_KEY.equals(name)) { throw new UnsupportedOperationException( - "KvEntityStore is not supported since this version. Please use RelationalEntityStore instead."); + "KvEntityStore is not supported since version 0.6.0. Please use RelationalEntityStore instead."); } try { diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestEntityKeyEncoding.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestEntityKeyEncoding.java index 1347506a2..a8c7a2ac3 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestEntityKeyEncoding.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestEntityKeyEncoding.java @@ -56,7 +56,7 @@ import org.junit.jupiter.params.provider.MethodSource; import org.mockito.Mockito; @TestInstance(Lifecycle.PER_CLASS) -@Disabled +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") public class TestEntityKeyEncoding { private Config getConfig() throws IOException { File baseDir = new File(System.getProperty("java.io.tmpdir")); diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvEntityStorage.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvEntityStorage.java index 2a525ec7a..28985c31e 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvEntityStorage.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvEntityStorage.java @@ -63,7 +63,7 @@ import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.mockito.Mockito; -@Disabled +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") public class TestKvEntityStorage extends TestEntityStorage { @BeforeEach @AfterEach diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvGarbageCollector.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvGarbageCollector.java index 79e66d91e..cef6e2e40 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvGarbageCollector.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvGarbageCollector.java @@ -64,7 +64,7 @@ import org.junit.jupiter.api.Test; import org.mockito.Mockito; @SuppressWarnings("DefaultCharset") -@Disabled +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") class TestKvGarbageCollector { public Config getConfig() throws IOException { Config config = Mockito.mock(Config.class); diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvNameMappingService.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvNameMappingService.java index 7b8314be9..f8cdd091b 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvNameMappingService.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestKvNameMappingService.java @@ -41,7 +41,7 @@ import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.mockito.Mockito; -@Disabled +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") public class TestKvNameMappingService { private Config getConfig() throws IOException { File baseDir = new File(System.getProperty("java.io.tmpdir")); diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestRocksDBKvBackend.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestRocksDBKvBackend.java index 588908ba2..e1d3d80b0 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestRocksDBKvBackend.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestRocksDBKvBackend.java @@ -30,11 +30,13 @@ import java.nio.file.Files; import java.util.List; import org.apache.commons.lang3.tuple.Pair; import org.junit.jupiter.api.Assertions; +import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.mockito.Mockito; import org.rocksdb.RocksDB; import org.rocksdb.RocksDBException; +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") public class TestRocksDBKvBackend { private KvBackend getKvBackEnd() throws IOException { diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestStorageVersion.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestStorageVersion.java index d54c405c0..fcf1eb03d 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestStorageVersion.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestStorageVersion.java @@ -41,7 +41,7 @@ import org.junit.jupiter.api.Disabled; import org.junit.jupiter.api.Test; import org.mockito.Mockito; -@Disabled +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") class TestStorageVersion { @Test diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionIdGenerator.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionIdGenerator.java index a2903bc4a..22e6b9f5e 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionIdGenerator.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionIdGenerator.java @@ -42,6 +42,7 @@ import org.mockito.Mockito; import org.slf4j.Logger; import org.slf4j.LoggerFactory; +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") public class TestTransactionIdGenerator { private static final Logger LOGGER = LoggerFactory.getLogger(TestTransactionalKvBackend.class); diff --git a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionalKvBackend.java b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionalKvBackend.java index 936fbc492..7514c290c 100644 --- a/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionalKvBackend.java +++ b/core/src/test/java/com/datastrato/gravitino/storage/kv/TestTransactionalKvBackend.java @@ -54,6 +54,7 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; @SuppressWarnings("DefaultCharset") +@Disabled("Gravitino will not support KV entity store since 0.6.0, so we disable this test.") class TestTransactionalKvBackend { private static final Logger LOGGER = LoggerFactory.getLogger(TestTransactionalKvBackend.class); diff --git a/gradle.properties b/gradle.properties index 16c85c2c8..6e7ecf468 100644 --- a/gradle.properties +++ b/gradle.properties @@ -40,6 +40,6 @@ defaultScalaVersion = 2.12 pythonVersion = 3.8 # skipDockerTests is used to skip the tests that require Docker to be running. -skipDockerTests = false +skipDockerTests = true # The default backend for the JDBC tests, you set it to h2 or mysql jdbcBackend = h2
