From 25bf2accb1a7ec685aca1508265f2dfa86a95667 Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Wed, 18 Dec 2024 14:51:38 +0100 Subject: [PATCH 01/13] Modify IcebergCatalogWrapper so it ignores Warehouse URI in case of a REST type catalog. --- .../gravitino/iceberg/common/ops/IcebergCatalogWrapper.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java index 05c9ee2a1eb..715a56970d3 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java @@ -82,7 +82,8 @@ public IcebergCatalogWrapper(IcebergConfig icebergConfig) { this.catalogBackend = IcebergCatalogBackend.valueOf( icebergConfig.get(IcebergConfig.CATALOG_BACKEND).toUpperCase(Locale.ROOT)); - if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend)) { + if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend) && + !IcebergCatalogBackend.REST.equals(catalogBackend)) { // check whether IcebergConfig.CATALOG_WAREHOUSE exists icebergConfig.get(IcebergConfig.CATALOG_WAREHOUSE); this.catalogUri = icebergConfig.get(IcebergConfig.CATALOG_URI); From ca56609e79f212c7726b38d14912ea8083aaba52 Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Wed, 18 Dec 2024 20:16:11 +0100 Subject: [PATCH 02/13] Modify the property WAREHOUSE in IcebergCatalogPropertiesMetadata to Optional --- .../lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java b/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java index 6d61a6220a3..79da375ef30 100644 --- a/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java +++ b/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java @@ -74,7 +74,7 @@ public class IcebergCatalogPropertiesMetadata extends BaseCatalogPropertiesMetad false /* reserved */), stringRequiredPropertyEntry( URI, "Iceberg catalog uri config", false /* immutable */, false /* hidden */), - stringRequiredPropertyEntry( + stringOptionalPropertyEntry( WAREHOUSE, "Iceberg catalog warehouse config", false /* immutable */, From 63237050b696b2f7a35af0733cdf168bd47251da Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 09:42:42 +0100 Subject: [PATCH 03/13] Fixing WAREHOUSE parameters in IcebergCatalogPropertiesMetadata --- .../lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java | 1 + 1 file changed, 1 insertion(+) diff --git a/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java b/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java index 79da375ef30..375edd600fb 100644 --- a/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java +++ b/catalogs/catalog-lakehouse-iceberg/src/main/java/org/apache/gravitino/catalog/lakehouse/iceberg/IcebergCatalogPropertiesMetadata.java @@ -78,6 +78,7 @@ public class IcebergCatalogPropertiesMetadata extends BaseCatalogPropertiesMetad WAREHOUSE, "Iceberg catalog warehouse config", false /* immutable */, + null, /* defaultValue */ false /* hidden */), stringOptionalPropertyEntry( IcebergConstants.IO_IMPL, From 12d517c5bc1af9bdb7463d1f14d875bfdae37da2 Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 10:14:17 +0100 Subject: [PATCH 04/13] Add a test that use the minimum amount of required parameter for the IcebergCatalog --- .../lakehouse/iceberg/TestIcebergCatalog.java | 47 +++++++++++++++++++ 1 file changed, 47 insertions(+) diff --git a/catalogs/catalog-lakehouse-iceberg/src/test/java/org/apache/gravitino/catalog/lakehouse/iceberg/TestIcebergCatalog.java b/catalogs/catalog-lakehouse-iceberg/src/test/java/org/apache/gravitino/catalog/lakehouse/iceberg/TestIcebergCatalog.java index 5c657197231..8ff70d39854 100644 --- a/catalogs/catalog-lakehouse-iceberg/src/test/java/org/apache/gravitino/catalog/lakehouse/iceberg/TestIcebergCatalog.java +++ b/catalogs/catalog-lakehouse-iceberg/src/test/java/org/apache/gravitino/catalog/lakehouse/iceberg/TestIcebergCatalog.java @@ -146,4 +146,51 @@ void testCatalogProperty() { throwable.getMessage().contains(IcebergCatalogPropertiesMetadata.CATALOG_BACKEND)); } } + + @Test + void testCatalogInstanciation() { + AuditInfo auditInfo = + AuditInfo.builder().withCreator("creator").withCreateTime(Instant.now()).build(); + + CatalogEntity entity = + CatalogEntity.builder() + .withId(1L) + .withName("catalog") + .withNamespace(Namespace.of("metalake")) + .withType(IcebergCatalog.Type.RELATIONAL) + .withProvider("iceberg") + .withAuditInfo(auditInfo) + .build(); + + Map conf = Maps.newHashMap(); + + try (IcebergCatalogOperations ops = new IcebergCatalogOperations()) { + ops.initialize(conf, entity.toCatalogInfo(), ICEBERG_PROPERTIES_METADATA); + Map map1 = Maps.newHashMap(); + map1.put(IcebergCatalogPropertiesMetadata.CATALOG_BACKEND, "test"); + PropertiesMetadata metadata = ICEBERG_PROPERTIES_METADATA.catalogPropertiesMetadata(); + Assertions.assertThrows( + IllegalArgumentException.class, + () -> { + PropertiesMetadataHelpers.validatePropertyForCreate(metadata, map1); + }); + + Map map2 = Maps.newHashMap(); + map2.put(IcebergCatalogPropertiesMetadata.CATALOG_BACKEND, "rest"); + map2.put(IcebergCatalogPropertiesMetadata.URI, "127.0.0.1"); + Assertions.assertDoesNotThrow( + () -> { + PropertiesMetadataHelpers.validatePropertyForCreate(metadata, map2); + }); + + Map map3 = Maps.newHashMap(); + Throwable throwable = + Assertions.assertThrows( + IllegalArgumentException.class, + () -> PropertiesMetadataHelpers.validatePropertyForCreate(metadata, map3)); + + Assertions.assertTrue( + throwable.getMessage().contains(IcebergCatalogPropertiesMetadata.CATALOG_BACKEND)); + } + } } From 436bbd20702a5e0595cf8922dc64873c7e40eca0 Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 12:04:26 +0100 Subject: [PATCH 05/13] Modify WAREHOUSE ConfigEntry as Optional in IcebergConfig\nModify WAREHOUSE checks for presence in creatingCatalogWrapper for Iceberg --- gradle.properties | 2 +- .../java/org/apache/gravitino/iceberg/common/IcebergConfig.java | 1 + .../gravitino/iceberg/common/ops/IcebergCatalogWrapper.java | 2 ++ 3 files changed, 4 insertions(+), 1 deletion(-) diff --git a/gradle.properties b/gradle.properties index cc1b9393018..a243690bc42 100644 --- a/gradle.properties +++ b/gradle.properties @@ -23,7 +23,7 @@ org.gradle.caching=true org.gradle.jvmargs=-Xmx4g # version that is going to be updated automatically by releases -version = 0.8.0-incubating-SNAPSHOT +version = 0.7.0-incubating-SNAPSHOT # sonatype credentials SONATYPE_USER = admin diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java index 2e7eb74e2f1..844f8112f94 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java @@ -65,6 +65,7 @@ public class IcebergConfig extends Config implements OverwriteDefaultConfig { .doc("Warehouse directory of catalog") .version(ConfigConstants.VERSION_0_2_0) .stringConf() + .setOptional() .checkValue(StringUtils::isNotBlank, ConfigConstants.NOT_BLANK_ERROR_MSG) .create(); diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java index 715a56970d3..12b97864880 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java @@ -86,6 +86,8 @@ public IcebergCatalogWrapper(IcebergConfig icebergConfig) { !IcebergCatalogBackend.REST.equals(catalogBackend)) { // check whether IcebergConfig.CATALOG_WAREHOUSE exists icebergConfig.get(IcebergConfig.CATALOG_WAREHOUSE); + } + if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend)){ this.catalogUri = icebergConfig.get(IcebergConfig.CATALOG_URI); } this.catalog = IcebergCatalogUtil.loadCatalogBackend(catalogBackend, icebergConfig); From 27a96babf7e4b059b17ef6b07cbd79b1b020fd70 Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 12:21:57 +0100 Subject: [PATCH 06/13] Modify WAREHOUSE constant config in IcebergConfig as Optional --- .../org/apache/gravitino/iceberg/common/IcebergConfig.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java index 844f8112f94..a05a019f544 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java @@ -65,9 +65,7 @@ public class IcebergConfig extends Config implements OverwriteDefaultConfig { .doc("Warehouse directory of catalog") .version(ConfigConstants.VERSION_0_2_0) .stringConf() - .setOptional() - .checkValue(StringUtils::isNotBlank, ConfigConstants.NOT_BLANK_ERROR_MSG) - .create(); + .createWithOptional(); public static final ConfigEntry CATALOG_URI = new ConfigBuilder(IcebergConstants.URI) From 09e9338dd9b3077b8b1478cefe5a5de06b497994 Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 12:54:48 +0100 Subject: [PATCH 07/13] Fixing WAREHOUSE configEntry declaration in IcebergConfig --- .../java/org/apache/gravitino/iceberg/common/IcebergConfig.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java index a05a019f544..c1284c5adbd 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java @@ -60,7 +60,7 @@ public class IcebergConfig extends Config implements OverwriteDefaultConfig { .stringConf() .create(); - public static final ConfigEntry CATALOG_WAREHOUSE = + public static final ConfigEntry> CATALOG_WAREHOUSE = new ConfigBuilder(IcebergConstants.WAREHOUSE) .doc("Warehouse directory of catalog") .version(ConfigConstants.VERSION_0_2_0) From ea70ac325bf1a03b02b5dec0421df534fc4da1dc Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 12:57:26 +0100 Subject: [PATCH 08/13] Fixing imports declarations in IcebergConfig --- .../java/org/apache/gravitino/iceberg/common/IcebergConfig.java | 1 + 1 file changed, 1 insertion(+) diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java index c1284c5adbd..1440760bb54 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java @@ -23,6 +23,7 @@ import com.google.common.collect.ImmutableMap; import java.util.Collections; import java.util.List; +import java.util.Optional; import java.util.Map; import org.apache.commons.lang3.StringUtils; import org.apache.gravitino.Config; From b1ace6d74613aea2560c75a91ed060ff653c219d Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 16:12:04 +0100 Subject: [PATCH 09/13] Fixing ConfigEntry instanciation for WAREHOUSE to non-optional, create, without value checks. --- .../org/apache/gravitino/iceberg/common/IcebergConfig.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java index 1440760bb54..7062b9e2467 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java @@ -61,12 +61,12 @@ public class IcebergConfig extends Config implements OverwriteDefaultConfig { .stringConf() .create(); - public static final ConfigEntry> CATALOG_WAREHOUSE = + public static final ConfigEntry CATALOG_WAREHOUSE = new ConfigBuilder(IcebergConstants.WAREHOUSE) .doc("Warehouse directory of catalog") .version(ConfigConstants.VERSION_0_2_0) .stringConf() - .createWithOptional(); + .create(); public static final ConfigEntry CATALOG_URI = new ConfigBuilder(IcebergConstants.URI) From c14c51049880f08bcd329e1c90b747aca0979f6d Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Thu, 19 Dec 2024 16:24:14 +0000 Subject: [PATCH 10/13] Correcting IcebergCatalogBackend for usage with non-required WAREHOUSE\nRemoving systematic value check for WAREHOUSE parameter in REST request. --- gradle.properties | 4 ++-- .../org/apache/gravitino/iceberg/common/IcebergConfig.java | 1 - .../gravitino/iceberg/common/ops/IcebergCatalogWrapper.java | 6 +++--- 3 files changed, 5 insertions(+), 6 deletions(-) diff --git a/gradle.properties b/gradle.properties index a243690bc42..b7cd1bb69ef 100644 --- a/gradle.properties +++ b/gradle.properties @@ -20,7 +20,7 @@ org.gradle.parallel=true # Build cache can be disabled with --no-build-cache option org.gradle.caching=true -org.gradle.jvmargs=-Xmx4g +org.gradle.jvmargs=-Xmx12g # version that is going to be updated automatically by releases version = 0.7.0-incubating-SNAPSHOT @@ -31,7 +31,7 @@ SONATYPE_PASSWORD = password # jdkVersion is used to specify the version of JDK to build and test Gravitino, current # supported version is 8, 11, and 17. -jdkVersion = 8 +jdkVersion = 17 # defaultScalaVersion is used to specify the version of Scala to build and test Gravitino defaultScalaVersion = 2.12 diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java index 7062b9e2467..60a7491b854 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/IcebergConfig.java @@ -23,7 +23,6 @@ import com.google.common.collect.ImmutableMap; import java.util.Collections; import java.util.List; -import java.util.Optional; import java.util.Map; import org.apache.commons.lang3.StringUtils; import org.apache.gravitino.Config; diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java index 12b97864880..cdcd13b7386 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java @@ -82,12 +82,12 @@ public IcebergCatalogWrapper(IcebergConfig icebergConfig) { this.catalogBackend = IcebergCatalogBackend.valueOf( icebergConfig.get(IcebergConfig.CATALOG_BACKEND).toUpperCase(Locale.ROOT)); - if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend) && - !IcebergCatalogBackend.REST.equals(catalogBackend)) { + if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend) + && !IcebergCatalogBackend.REST.equals(catalogBackend)) { // check whether IcebergConfig.CATALOG_WAREHOUSE exists icebergConfig.get(IcebergConfig.CATALOG_WAREHOUSE); } - if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend)){ + if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend)) { this.catalogUri = icebergConfig.get(IcebergConfig.CATALOG_URI); } this.catalog = IcebergCatalogUtil.loadCatalogBackend(catalogBackend, icebergConfig); From 286fdbec96ee7806beca4ae3473640436aef75ee Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Sat, 21 Dec 2024 19:53:47 +0100 Subject: [PATCH 11/13] Re-establishing original gradle.properties\nWrapping out of style elements in Properties Metadata for Iceberg Catalog\nTesting that warehouse is not blank if necessary for some type of catalog for Iceberg. --- gradle.properties | 6 +++--- .../gravitino/iceberg/common/ops/IcebergCatalogWrapper.java | 5 ++++- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/gradle.properties b/gradle.properties index b7cd1bb69ef..458b0948c2a 100644 --- a/gradle.properties +++ b/gradle.properties @@ -20,7 +20,7 @@ org.gradle.parallel=true # Build cache can be disabled with --no-build-cache option org.gradle.caching=true -org.gradle.jvmargs=-Xmx12g +org.gradle.jvmargs=-Xmx4g # version that is going to be updated automatically by releases version = 0.7.0-incubating-SNAPSHOT @@ -31,13 +31,13 @@ SONATYPE_PASSWORD = password # jdkVersion is used to specify the version of JDK to build and test Gravitino, current # supported version is 8, 11, and 17. -jdkVersion = 17 +jdkVersion = 8 # defaultScalaVersion is used to specify the version of Scala to build and test Gravitino defaultScalaVersion = 2.12 # pythonVersion is used to specify the version of Python to build and test Gravitino python client. -pythonVersion = 3.8 +pythonVersion = 3.11 # skipDockerTests is used to skip the tests that require Docker to be running. skipDockerTests = true diff --git a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java index cdcd13b7386..0ed62b26f7f 100644 --- a/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java +++ b/iceberg/iceberg-common/src/main/java/org/apache/gravitino/iceberg/common/ops/IcebergCatalogWrapper.java @@ -29,6 +29,7 @@ import java.util.function.Supplier; import lombok.Getter; import lombok.Setter; +import org.apache.commons.lang3.StringUtils; import org.apache.gravitino.catalog.lakehouse.iceberg.IcebergConstants; import org.apache.gravitino.iceberg.common.IcebergCatalogBackend; import org.apache.gravitino.iceberg.common.IcebergConfig; @@ -85,7 +86,9 @@ public IcebergCatalogWrapper(IcebergConfig icebergConfig) { if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend) && !IcebergCatalogBackend.REST.equals(catalogBackend)) { // check whether IcebergConfig.CATALOG_WAREHOUSE exists - icebergConfig.get(IcebergConfig.CATALOG_WAREHOUSE); + if (StringUtils.isBlank(icebergConfig.get(IcebergConfig.CATALOG_WAREHOUSE))) { + throw new IllegalArgumentException("The 'warehouse' parameter must have a value."); + } } if (!IcebergCatalogBackend.MEMORY.equals(catalogBackend)) { this.catalogUri = icebergConfig.get(IcebergConfig.CATALOG_URI); From 7f8b3cfaee3fd6460e9d1907c11e0dde0d7b61fb Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Sun, 22 Dec 2024 09:43:04 +0100 Subject: [PATCH 12/13] Inverting past manual changes to gradle.properties version software version attribute. --- gradle.properties | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gradle.properties b/gradle.properties index 458b0948c2a..a8110532c98 100644 --- a/gradle.properties +++ b/gradle.properties @@ -23,7 +23,7 @@ org.gradle.caching=true org.gradle.jvmargs=-Xmx4g # version that is going to be updated automatically by releases -version = 0.7.0-incubating-SNAPSHOT +version = 0.8.0-incubating-SNAPSHOT # sonatype credentials SONATYPE_USER = admin From 44667a516a13312adaafef6dab28c6e9ade47d72 Mon Sep 17 00:00:00 2001 From: Faisal Salhi Date: Sun, 22 Dec 2024 12:55:25 +0100 Subject: [PATCH 13/13] Reverting changes made to pythonVersion in gradle properties. --- gradle.properties | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gradle.properties b/gradle.properties index a8110532c98..cc1b9393018 100644 --- a/gradle.properties +++ b/gradle.properties @@ -37,7 +37,7 @@ jdkVersion = 8 defaultScalaVersion = 2.12 # pythonVersion is used to specify the version of Python to build and test Gravitino python client. -pythonVersion = 3.11 +pythonVersion = 3.8 # skipDockerTests is used to skip the tests that require Docker to be running. skipDockerTests = true