diff --git a/docs/src/main/sphinx/connector/metastores.md b/docs/src/main/sphinx/connector/metastores.md index 739cb5c8fafa4..950e2248055cd 100644 --- a/docs/src/main/sphinx/connector/metastores.md +++ b/docs/src/main/sphinx/connector/metastores.md @@ -295,7 +295,7 @@ described with the following additional property: - Skip archiving an old table version when creating a new version in a commit. See [AWS Glue Skip Archive](https://iceberg.apache.org/docs/latest/aws/#skip-archive). - - `false` + - `true` ::: ## Iceberg-specific metastores diff --git a/plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/catalog/glue/IcebergGlueCatalogConfig.java b/plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/catalog/glue/IcebergGlueCatalogConfig.java index 1f8ba21a56c3e..156f48acebc0a 100644 --- a/plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/catalog/glue/IcebergGlueCatalogConfig.java +++ b/plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/catalog/glue/IcebergGlueCatalogConfig.java @@ -19,7 +19,7 @@ public class IcebergGlueCatalogConfig { private boolean cacheTableMetadata = true; - private boolean skipArchive; + private boolean skipArchive = true; public boolean isCacheTableMetadata() { diff --git a/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogConfig.java b/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogConfig.java index 3c21ffb695926..c6b3293401540 100644 --- a/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogConfig.java +++ b/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogConfig.java @@ -29,7 +29,7 @@ public void testDefaults() { assertRecordedDefaults(recordDefaults(IcebergGlueCatalogConfig.class) .setCacheTableMetadata(true) - .setSkipArchive(false)); + .setSkipArchive(true)); } @Test @@ -37,12 +37,12 @@ public void testExplicitPropertyMapping() { Map properties = ImmutableMap.builder() .put("iceberg.glue.cache-table-metadata", "false") - .put("iceberg.glue.skip-archive", "true") + .put("iceberg.glue.skip-archive", "false") .buildOrThrow(); IcebergGlueCatalogConfig expected = new IcebergGlueCatalogConfig() .setCacheTableMetadata(false) - .setSkipArchive(true); + .setSkipArchive(false); assertFullMapping(properties, expected); } diff --git a/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogSkipArchive.java b/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogSkipArchive.java index 36638fbb28ce7..629e97e1a8477 100644 --- a/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogSkipArchive.java +++ b/plugin/trino-iceberg/src/test/java/io/trino/plugin/iceberg/catalog/glue/TestIcebergGlueCatalogSkipArchive.java @@ -82,7 +82,6 @@ protected QueryRunner createQueryRunner() .setIcebergProperties( ImmutableMap.builder() .put("iceberg.catalog.type", "glue") - .put("iceberg.glue.skip-archive", "true") .put("hive.metastore.glue.default-warehouse-dir", schemaDirectory.getAbsolutePath()) .buildOrThrow()) .setSchemaInitializer(