Core: Move iceberg-parquet files to iceberg-core

- Since core module need to write stats in parquet format, to avoid circular dependency, move all the files from iceberg-parquet module to iceberg code. - `TestParquetReadProjection` used to duplicate the test code of iceberg-api module's `TestReadProjection`. Removed the duplicate class and instead directly extend the original class from iceberg-api module. - Update TestParquetReadProjection to skip empty struct testcases as only Avro readers supports it. The testcases are now common for both Avro and Parquet readers.
apache · Jan 29, 2025 · a8f416f · a8f416f
1 parent e89798e
commit a8f416f
Show file tree

Hide file tree

Showing 74 changed files with 12 additions and 53 deletions.
diff --git a/README.md b/README.md
@@ -63,7 +63,6 @@ Iceberg table support is organized in library modules:
 * `iceberg-common` contains utility classes used in other modules
 * `iceberg-api` contains the public Iceberg API
 * `iceberg-core` contains implementations of the Iceberg API and support for Avro data files, **this is what processing engines should depend on**
-* `iceberg-parquet` is an optional module for working with tables backed by Parquet files
 * `iceberg-arrow` is an optional module for reading Parquet into Arrow memory
 * `iceberg-orc` is an optional module for working with tables backed by ORC files
 * `iceberg-hive-metastore` is an implementation of Iceberg tables backed by the Hive metastore Thrift client

diff --git a/build.gradle b/build.gradle
@@ -111,7 +111,7 @@ if (file("${rootDir}/iceberg-build.properties").exists()) {
 }
 
 def projectVersion = getProjectVersion()
-final REVAPI_PROJECTS = ["iceberg-api", "iceberg-core", "iceberg-parquet", "iceberg-orc", "iceberg-common", "iceberg-data"]
+final REVAPI_PROJECTS = ["iceberg-api", "iceberg-core", "iceberg-orc", "iceberg-common", "iceberg-data"]
 
 allprojects {
   group = "org.apache.iceberg"
@@ -157,7 +157,7 @@ subprojects {
         rootTask.finalizedBy showDeprecationRulesOnRevApiFailure
       }
     }
-    
+
     tasks.named("revapiAnalyze").configure {
       dependsOn(":iceberg-common:jar")
     }
@@ -353,6 +353,13 @@ project(':iceberg-core') {
       exclude group: 'org.slf4j', module: 'slf4j-log4j12'
     }
 
+    implementation(libs.parquet.avro) {
+      exclude group: 'org.apache.avro', module: 'avro'
+      // already shaded by Parquet
+      exclude group: 'it.unimi.dsi'
+      exclude group: 'org.codehaus.jackson'
+    }
+
     testImplementation libs.jetty.servlet
     testImplementation libs.jakarta.servlet
     testImplementation libs.jetty.server
@@ -371,7 +378,6 @@ project(':iceberg-data') {
     implementation project(path: ':iceberg-bundled-guava', configuration: 'shadow')
     api project(':iceberg-api')
     implementation project(':iceberg-core')
-    compileOnly project(':iceberg-parquet')
     compileOnly project(':iceberg-orc')
     compileOnly(libs.hadoop2.common) {
       exclude group: 'commons-beanutils'
@@ -564,7 +570,6 @@ project(':iceberg-delta-lake') {
     api project(':iceberg-api')
     implementation project(':iceberg-common')
     implementation project(':iceberg-core')
-    implementation project(':iceberg-parquet')
     implementation platform(libs.jackson.bom)
     implementation libs.jackson.databind
     annotationProcessor libs.immutables.value
@@ -770,33 +775,6 @@ project(':iceberg-orc') {
   }
 }
 
-project(':iceberg-parquet') {
-  test {
-    useJUnitPlatform()
-  }
-  dependencies {
-    implementation project(path: ':iceberg-bundled-guava', configuration: 'shadow')
-    api project(':iceberg-api')
-    implementation project(':iceberg-core')
-    implementation project(':iceberg-common')
-
-    implementation(libs.parquet.avro) {
-      exclude group: 'org.apache.avro', module: 'avro'
-      // already shaded by Parquet
-      exclude group: 'it.unimi.dsi'
-      exclude group: 'org.codehaus.jackson'
-    }
-
-    compileOnly libs.avro.avro
-    compileOnly(libs.hadoop2.client) {
-      exclude group: 'org.apache.avro', module: 'avro'
-    }
-
-    testImplementation project(path: ':iceberg-api', configuration: 'testArtifacts')
-    testImplementation project(path: ':iceberg-core', configuration: 'testArtifacts')
-  }
-}
-
 project(':iceberg-arrow') {
   test {
     useJUnitPlatform()
@@ -805,7 +783,6 @@ project(':iceberg-arrow') {
     implementation project(path: ':iceberg-bundled-guava', configuration: 'shadow')
     api project(':iceberg-api')
     implementation project(':iceberg-core')
-    implementation project(':iceberg-parquet')
 
     implementation(libs.arrow.vector) {
       exclude group: 'io.netty', module: 'netty-buffer'
@@ -897,7 +874,9 @@ project(':iceberg-snowflake') {
   }
 
   dependencies {
-    implementation project(':iceberg-core')
+    implementation(project(':iceberg-core')) {
+      exclude group: 'com.github.luben', module: 'zstd-jni'
+    }
     implementation project(':iceberg-common')
     implementation project(path: ':iceberg-bundled-guava', configuration: 'shadow')
     implementation platform(libs.jackson.bom)

diff --git a/...berg/data/parquet/BaseParquetReaders.java → ...berg/data/parquet/BaseParquetReaders.java b/...berg/data/parquet/BaseParquetReaders.java → ...berg/data/parquet/BaseParquetReaders.java
diff --git a/...eberg/data/parquet/BaseParquetWriter.java → ...eberg/data/parquet/BaseParquetWriter.java b/...eberg/data/parquet/BaseParquetWriter.java → ...eberg/data/parquet/BaseParquetWriter.java
diff --git a/...g/data/parquet/GenericParquetReaders.java → ...g/data/parquet/GenericParquetReaders.java b/...g/data/parquet/GenericParquetReaders.java → ...g/data/parquet/GenericParquetReaders.java
diff --git a/...rg/data/parquet/GenericParquetWriter.java → ...rg/data/parquet/GenericParquetWriter.java b/...rg/data/parquet/GenericParquetWriter.java → ...rg/data/parquet/GenericParquetWriter.java
diff --git a/.../iceberg/data/parquet/InternalReader.java → .../iceberg/data/parquet/InternalReader.java b/.../iceberg/data/parquet/InternalReader.java → .../iceberg/data/parquet/InternalReader.java
diff --git a/.../iceberg/data/parquet/InternalWriter.java → .../iceberg/data/parquet/InternalWriter.java b/.../iceberg/data/parquet/InternalWriter.java → .../iceberg/data/parquet/InternalWriter.java
diff --git a/...che/iceberg/parquet/ApplyNameMapping.java → ...che/iceberg/parquet/ApplyNameMapping.java b/...che/iceberg/parquet/ApplyNameMapping.java → ...che/iceberg/parquet/ApplyNameMapping.java
diff --git a/...e/iceberg/parquet/BaseColumnIterator.java → ...e/iceberg/parquet/BaseColumnIterator.java b/...e/iceberg/parquet/BaseColumnIterator.java → ...e/iceberg/parquet/BaseColumnIterator.java
diff --git a/...che/iceberg/parquet/BasePageIterator.java → ...che/iceberg/parquet/BasePageIterator.java b/...che/iceberg/parquet/BasePageIterator.java → ...che/iceberg/parquet/BasePageIterator.java
diff --git a/...pache/iceberg/parquet/ColumnIterator.java → ...pache/iceberg/parquet/ColumnIterator.java b/...pache/iceberg/parquet/ColumnIterator.java → ...pache/iceberg/parquet/ColumnIterator.java
diff --git a/.../apache/iceberg/parquet/ColumnWriter.java → .../apache/iceberg/parquet/ColumnWriter.java b/.../apache/iceberg/parquet/ColumnWriter.java → .../apache/iceberg/parquet/ColumnWriter.java
diff --git a/...he/iceberg/parquet/MessageTypeToType.java → ...he/iceberg/parquet/MessageTypeToType.java b/...he/iceberg/parquet/MessageTypeToType.java → ...he/iceberg/parquet/MessageTypeToType.java
diff --git a/.../apache/iceberg/parquet/PageIterator.java → .../apache/iceberg/parquet/PageIterator.java b/.../apache/iceberg/parquet/PageIterator.java → .../apache/iceberg/parquet/PageIterator.java
diff --git a/...a/org/apache/iceberg/parquet/Parquet.java → ...a/org/apache/iceberg/parquet/Parquet.java b/...a/org/apache/iceberg/parquet/Parquet.java → ...a/org/apache/iceberg/parquet/Parquet.java
diff --git a/...g/apache/iceberg/parquet/ParquetAvro.java → ...g/apache/iceberg/parquet/ParquetAvro.java b/...g/apache/iceberg/parquet/ParquetAvro.java → ...g/apache/iceberg/parquet/ParquetAvro.java
diff --git a/...he/iceberg/parquet/ParquetAvroReader.java → ...he/iceberg/parquet/ParquetAvroReader.java b/...he/iceberg/parquet/ParquetAvroReader.java → ...he/iceberg/parquet/ParquetAvroReader.java
diff --git a/...berg/parquet/ParquetAvroValueReaders.java → ...berg/parquet/ParquetAvroValueReaders.java b/...berg/parquet/ParquetAvroValueReaders.java → ...berg/parquet/ParquetAvroValueReaders.java
diff --git a/...he/iceberg/parquet/ParquetAvroWriter.java → ...he/iceberg/parquet/ParquetAvroWriter.java b/...he/iceberg/parquet/ParquetAvroWriter.java → ...he/iceberg/parquet/ParquetAvroWriter.java
diff --git a/...g/parquet/ParquetBloomRowGroupFilter.java → ...g/parquet/ParquetBloomRowGroupFilter.java b/...g/parquet/ParquetBloomRowGroupFilter.java → ...g/parquet/ParquetBloomRowGroupFilter.java
diff --git a/.../iceberg/parquet/ParquetCodecFactory.java → .../iceberg/parquet/ParquetCodecFactory.java b/.../iceberg/parquet/ParquetCodecFactory.java → .../iceberg/parquet/ParquetCodecFactory.java
diff --git a/...e/iceberg/parquet/ParquetConversions.java → ...e/iceberg/parquet/ParquetConversions.java b/...e/iceberg/parquet/ParquetConversions.java → ...e/iceberg/parquet/ParquetConversions.java
diff --git a/...quet/ParquetDictionaryRowGroupFilter.java → ...quet/ParquetDictionaryRowGroupFilter.java b/...quet/ParquetDictionaryRowGroupFilter.java → ...quet/ParquetDictionaryRowGroupFilter.java
diff --git a/...pache/iceberg/parquet/ParquetFilters.java → ...pache/iceberg/parquet/ParquetFilters.java b/...pache/iceberg/parquet/ParquetFilters.java → ...pache/iceberg/parquet/ParquetFilters.java
diff --git a/...org/apache/iceberg/parquet/ParquetIO.java → ...org/apache/iceberg/parquet/ParquetIO.java b/...org/apache/iceberg/parquet/ParquetIO.java → ...org/apache/iceberg/parquet/ParquetIO.java
diff --git a/...ache/iceberg/parquet/ParquetIterable.java → ...ache/iceberg/parquet/ParquetIterable.java b/...ache/iceberg/parquet/ParquetIterable.java → ...ache/iceberg/parquet/ParquetIterable.java
diff --git a/...parquet/ParquetMetricsRowGroupFilter.java → ...parquet/ParquetMetricsRowGroupFilter.java b/...parquet/ParquetMetricsRowGroupFilter.java → ...parquet/ParquetMetricsRowGroupFilter.java
diff --git a/...e/iceberg/parquet/ParquetReadSupport.java → ...e/iceberg/parquet/ParquetReadSupport.java b/...e/iceberg/parquet/ParquetReadSupport.java → ...e/iceberg/parquet/ParquetReadSupport.java
diff --git a/...apache/iceberg/parquet/ParquetReader.java → ...apache/iceberg/parquet/ParquetReader.java b/...apache/iceberg/parquet/ParquetReader.java → ...apache/iceberg/parquet/ParquetReader.java
diff --git a/...he/iceberg/parquet/ParquetSchemaUtil.java → ...he/iceberg/parquet/ParquetSchemaUtil.java b/...he/iceberg/parquet/ParquetSchemaUtil.java → ...he/iceberg/parquet/ParquetSchemaUtil.java
diff --git a/...e/iceberg/parquet/ParquetTypeVisitor.java → ...e/iceberg/parquet/ParquetTypeVisitor.java b/...e/iceberg/parquet/ParquetTypeVisitor.java → ...e/iceberg/parquet/ParquetTypeVisitor.java
diff --git a/...g/apache/iceberg/parquet/ParquetUtil.java → ...g/apache/iceberg/parquet/ParquetUtil.java b/...g/apache/iceberg/parquet/ParquetUtil.java → ...g/apache/iceberg/parquet/ParquetUtil.java
diff --git a/...e/iceberg/parquet/ParquetValueReader.java → ...e/iceberg/parquet/ParquetValueReader.java b/...e/iceberg/parquet/ParquetValueReader.java → ...e/iceberg/parquet/ParquetValueReader.java
diff --git a/.../iceberg/parquet/ParquetValueReaders.java → .../iceberg/parquet/ParquetValueReaders.java b/.../iceberg/parquet/ParquetValueReaders.java → .../iceberg/parquet/ParquetValueReaders.java
diff --git a/...e/iceberg/parquet/ParquetValueWriter.java → ...e/iceberg/parquet/ParquetValueWriter.java b/...e/iceberg/parquet/ParquetValueWriter.java → ...e/iceberg/parquet/ParquetValueWriter.java
diff --git a/.../iceberg/parquet/ParquetValueWriters.java → .../iceberg/parquet/ParquetValueWriters.java b/.../iceberg/parquet/ParquetValueWriters.java → .../iceberg/parquet/ParquetValueWriters.java
diff --git a/.../iceberg/parquet/ParquetWriteAdapter.java → .../iceberg/parquet/ParquetWriteAdapter.java b/.../iceberg/parquet/ParquetWriteAdapter.java → .../iceberg/parquet/ParquetWriteAdapter.java
diff --git a/.../iceberg/parquet/ParquetWriteSupport.java → .../iceberg/parquet/ParquetWriteSupport.java b/.../iceberg/parquet/ParquetWriteSupport.java → .../iceberg/parquet/ParquetWriteSupport.java
diff --git a/...apache/iceberg/parquet/ParquetWriter.java → ...apache/iceberg/parquet/ParquetWriter.java b/...apache/iceberg/parquet/ParquetWriter.java → ...apache/iceberg/parquet/ParquetWriter.java
diff --git a/.../apache/iceberg/parquet/PruneColumns.java → .../apache/iceberg/parquet/PruneColumns.java b/.../apache/iceberg/parquet/PruneColumns.java → .../apache/iceberg/parquet/PruneColumns.java
diff --git a/.../org/apache/iceberg/parquet/ReadConf.java → .../org/apache/iceberg/parquet/ReadConf.java b/.../org/apache/iceberg/parquet/ReadConf.java → .../org/apache/iceberg/parquet/ReadConf.java
diff --git a/...org/apache/iceberg/parquet/RemoveIds.java → ...org/apache/iceberg/parquet/RemoveIds.java b/...org/apache/iceberg/parquet/RemoveIds.java → ...org/apache/iceberg/parquet/RemoveIds.java
diff --git a/...pache/iceberg/parquet/TripleIterator.java → ...pache/iceberg/parquet/TripleIterator.java b/...pache/iceberg/parquet/TripleIterator.java → ...pache/iceberg/parquet/TripleIterator.java
diff --git a/.../apache/iceberg/parquet/TripleWriter.java → .../apache/iceberg/parquet/TripleWriter.java b/.../apache/iceberg/parquet/TripleWriter.java → .../apache/iceberg/parquet/TripleWriter.java
diff --git a/...he/iceberg/parquet/TypeToMessageType.java → ...he/iceberg/parquet/TypeToMessageType.java b/...he/iceberg/parquet/TypeToMessageType.java → ...he/iceberg/parquet/TypeToMessageType.java
diff --git a/...ceberg/parquet/TypeWithSchemaVisitor.java → ...ceberg/parquet/TypeWithSchemaVisitor.java b/...ceberg/parquet/TypeWithSchemaVisitor.java → ...ceberg/parquet/TypeWithSchemaVisitor.java
diff --git a/.../iceberg/parquet/ValuesAsBytesReader.java → .../iceberg/parquet/ValuesAsBytesReader.java b/.../iceberg/parquet/ValuesAsBytesReader.java → .../iceberg/parquet/ValuesAsBytesReader.java
diff --git a/...berg/parquet/VectorizedParquetReader.java → ...berg/parquet/VectorizedParquetReader.java b/...berg/parquet/VectorizedParquetReader.java → ...berg/parquet/VectorizedParquetReader.java
diff --git a/...che/iceberg/parquet/VectorizedReader.java → ...che/iceberg/parquet/VectorizedReader.java b/...che/iceberg/parquet/VectorizedReader.java → ...che/iceberg/parquet/VectorizedReader.java
diff --git a/...eberg/avro/TestParquetReadProjection.java → ...eberg/avro/TestParquetReadProjection.java b/...eberg/avro/TestParquetReadProjection.java → ...eberg/avro/TestParquetReadProjection.java
diff --git a/...berg/parquet/ParquetWritingTestUtils.java → ...berg/parquet/ParquetWritingTestUtils.java b/...berg/parquet/ParquetWritingTestUtils.java → ...berg/parquet/ParquetWritingTestUtils.java
diff --git a/...berg/parquet/TestBloomRowGroupFilter.java → ...berg/parquet/TestBloomRowGroupFilter.java b/...berg/parquet/TestBloomRowGroupFilter.java → ...berg/parquet/TestBloomRowGroupFilter.java
diff --git a/...erg/parquet/TestCDHParquetStatistics.java → ...erg/parquet/TestCDHParquetStatistics.java b/...erg/parquet/TestCDHParquetStatistics.java → ...erg/parquet/TestCDHParquetStatistics.java
diff --git a/...parquet/TestDictionaryRowGroupFilter.java → ...parquet/TestDictionaryRowGroupFilter.java b/...parquet/TestDictionaryRowGroupFilter.java → ...parquet/TestDictionaryRowGroupFilter.java
diff --git a/.../iceberg/parquet/TestInternalParquet.java → .../iceberg/parquet/TestInternalParquet.java b/.../iceberg/parquet/TestInternalParquet.java → .../iceberg/parquet/TestInternalParquet.java
diff --git a/...g/apache/iceberg/parquet/TestParquet.java → ...g/apache/iceberg/parquet/TestParquet.java b/...g/apache/iceberg/parquet/TestParquet.java → ...g/apache/iceberg/parquet/TestParquet.java
diff --git a/...ceberg/parquet/TestParquetDataWriter.java → ...ceberg/parquet/TestParquetDataWriter.java b/...ceberg/parquet/TestParquetDataWriter.java → ...ceberg/parquet/TestParquetDataWriter.java
diff --git a/...erg/parquet/TestParquetDeleteWriters.java → ...erg/parquet/TestParquetDeleteWriters.java b/...erg/parquet/TestParquetDeleteWriters.java → ...erg/parquet/TestParquetDeleteWriters.java
diff --git a/...ceberg/parquet/TestParquetEncryption.java → ...ceberg/parquet/TestParquetEncryption.java b/...ceberg/parquet/TestParquetEncryption.java → ...ceberg/parquet/TestParquetEncryption.java
diff --git a/...ceberg/parquet/TestParquetSchemaUtil.java → ...ceberg/parquet/TestParquetSchemaUtil.java b/...ceberg/parquet/TestParquetSchemaUtil.java → ...ceberg/parquet/TestParquetSchemaUtil.java
diff --git a/...che/iceberg/parquet/TestPruneColumns.java → ...che/iceberg/parquet/TestPruneColumns.java b/...che/iceberg/parquet/TestPruneColumns.java → ...che/iceberg/parquet/TestPruneColumns.java
diff --git a/docs/docs/api.md b/docs/docs/api.md
@@ -239,7 +239,6 @@ Iceberg table support is organized in library modules:
 * `iceberg-arrow` is an implementation of the Iceberg type system for reading and writing data stored in Iceberg tables using Apache Arrow as the in-memory data format
 * `iceberg-aws` contains implementations of the Iceberg API to be used with tables stored on AWS S3 and/or for tables defined using the AWS Glue data catalog
 * `iceberg-core` contains implementations of the Iceberg API and support for Avro data files, **this is what processing engines should depend on**
-* `iceberg-parquet` is an optional module for working with tables backed by Parquet files
 * `iceberg-orc` is an optional module for working with tables backed by ORC files (*experimental*)
 * `iceberg-hive-metastore` is an implementation of Iceberg tables backed by the Hive metastore Thrift client
 

diff --git a/flink/v1.18/build.gradle b/flink/v1.18/build.gradle
@@ -29,7 +29,6 @@ project(":iceberg-flink:iceberg-flink-${flinkMajorVersion}") {
     implementation project(':iceberg-core')
     api project(':iceberg-data')
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
     implementation project(':iceberg-hive-metastore')
 
     compileOnly libs.flink118.avro

diff --git a/flink/v1.19/build.gradle b/flink/v1.19/build.gradle
@@ -29,7 +29,6 @@ project(":iceberg-flink:iceberg-flink-${flinkMajorVersion}") {
     implementation project(':iceberg-core')
     api project(':iceberg-data')
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
     implementation project(':iceberg-hive-metastore')
 
     compileOnly libs.flink119.avro

diff --git a/flink/v1.20/build.gradle b/flink/v1.20/build.gradle
@@ -29,7 +29,6 @@ project(":iceberg-flink:iceberg-flink-${flinkMajorVersion}") {
     implementation project(':iceberg-core')
     api project(':iceberg-data')
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
     implementation project(':iceberg-hive-metastore')
 
     compileOnly libs.flink120.avro

diff --git a/kafka-connect/build.gradle b/kafka-connect/build.gradle
@@ -49,7 +49,6 @@ project(':iceberg-kafka-connect:iceberg-kafka-connect') {
     compileOnly libs.kafka.connect.json
 
     testImplementation libs.hadoop3.client
-    testRuntimeOnly project(':iceberg-parquet')
     testRuntimeOnly project(':iceberg-orc')
   }
 
@@ -108,7 +107,6 @@ project(':iceberg-kafka-connect:iceberg-kafka-connect-runtime') {
       exclude group: 'org.eclipse.jetty'
     }
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
 
     implementation project(':iceberg-aws')
     implementation platform(libs.awssdk.bom)

diff --git a/mr/build.gradle b/mr/build.gradle
@@ -35,7 +35,6 @@ project(':iceberg-mr') {
     api project(':iceberg-data')
     implementation project(':iceberg-hive-metastore')
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
 
     compileOnly(libs.hadoop2.client) {
       exclude group: 'org.apache.avro', module: 'avro'

diff --git a/settings.gradle b/settings.gradle
@@ -30,7 +30,6 @@ include 'azure'
 include 'azure-bundle'
 include 'orc'
 include 'arrow'
-include 'parquet'
 include 'bundled-guava'
 include 'spark'
 include 'hive-metastore'
@@ -55,7 +54,6 @@ project(':azure').name = 'iceberg-azure'
 project(':azure-bundle').name = 'iceberg-azure-bundle'
 project(':orc').name = 'iceberg-orc'
 project(':arrow').name = 'iceberg-arrow'
-project(':parquet').name = 'iceberg-parquet'
 project(':bundled-guava').name = 'iceberg-bundled-guava'
 project(':spark').name = 'iceberg-spark'
 project(':hive-metastore').name = 'iceberg-hive-metastore'

diff --git a/site/docs/contribute.md b/site/docs/contribute.md
@@ -108,7 +108,6 @@ Iceberg table support is organized in library modules:
 * `iceberg-common` contains utility classes used in other modules
 * `iceberg-api` contains the public Iceberg API
 * `iceberg-core` contains implementations of the Iceberg API and support for Avro data files, **this is what processing engines should depend on**
-* `iceberg-parquet` is an optional module for working with tables backed by Parquet files
 * `iceberg-arrow` is an optional module for reading Parquet into Arrow memory
 * `iceberg-orc` is an optional module for working with tables backed by ORC files
 * `iceberg-hive-metastore` is an implementation of Iceberg tables backed by the Hive metastore Thrift client
@@ -155,7 +154,6 @@ __Modules__
 `iceberg-core`
 `iceberg-data`
 `iceberg-orc`
-`iceberg-parquet`
 
 Changes to public interfaces and classes in the subprojects listed above require a deprecation cycle of one minor
 release.

diff --git a/site/docs/releases.md b/site/docs/releases.md
@@ -54,8 +54,6 @@ dependencies {
 }
 ```
 
-You may also want to include `iceberg-parquet` for Parquet file support.
-
 ### Maven
 
 To add a dependency on Iceberg in Maven, add the following to your `pom.xml`:

diff --git a/spark/v3.3/build.gradle b/spark/v3.3/build.gradle
@@ -56,7 +56,6 @@ project(":iceberg-spark:iceberg-spark-${sparkMajorVersion}_${scalaVersion}") {
     implementation project(':iceberg-core')
     implementation project(':iceberg-data')
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
     implementation project(':iceberg-arrow')
     implementation("org.scala-lang.modules:scala-collection-compat_${scalaVersion}:${libs.versions.scala.collection.compat.get()}")
 
@@ -157,7 +156,6 @@ project(":iceberg-spark:iceberg-spark-extensions-${sparkMajorVersion}_${scalaVer
     compileOnly libs.errorprone.annotations
 
     testImplementation project(path: ':iceberg-data')
-    testImplementation project(path: ':iceberg-parquet')
     testImplementation project(path: ':iceberg-hive-metastore')
     testImplementation project(path: ':iceberg-api', configuration: 'testArtifacts')
     testImplementation project(path: ':iceberg-hive-metastore', configuration: 'testArtifacts')

diff --git a/spark/v3.4/build.gradle b/spark/v3.4/build.gradle
@@ -56,7 +56,6 @@ project(":iceberg-spark:iceberg-spark-${sparkMajorVersion}_${scalaVersion}") {
     implementation project(':iceberg-core')
     implementation project(':iceberg-data')
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
     implementation project(':iceberg-arrow')
     implementation("org.scala-lang.modules:scala-collection-compat_${scalaVersion}:${libs.versions.scala.collection.compat.get()}")
     implementation("org.apache.datasketches:datasketches-java:${libs.versions.datasketches.get()}")
@@ -164,7 +163,6 @@ project(":iceberg-spark:iceberg-spark-extensions-${sparkMajorVersion}_${scalaVer
     compileOnly libs.errorprone.annotations
 
     testImplementation project(path: ':iceberg-data')
-    testImplementation project(path: ':iceberg-parquet')
     testImplementation project(path: ':iceberg-hive-metastore')
     testImplementation project(path: ':iceberg-data', configuration: 'testArtifacts')
     testImplementation project(path: ':iceberg-api', configuration: 'testArtifacts')

diff --git a/spark/v3.5/build.gradle b/spark/v3.5/build.gradle
@@ -56,7 +56,6 @@ project(":iceberg-spark:iceberg-spark-${sparkMajorVersion}_${scalaVersion}") {
     implementation project(':iceberg-core')
     implementation project(':iceberg-data')
     implementation project(':iceberg-orc')
-    implementation project(':iceberg-parquet')
     implementation project(':iceberg-arrow')
     implementation("org.scala-lang.modules:scala-collection-compat_${scalaVersion}:${libs.versions.scala.collection.compat.get()}")
     implementation("org.apache.datasketches:datasketches-java:${libs.versions.datasketches.get()}")
@@ -163,7 +162,6 @@ project(":iceberg-spark:iceberg-spark-extensions-${sparkMajorVersion}_${scalaVer
     compileOnly libs.errorprone.annotations
 
     testImplementation project(path: ':iceberg-data')
-    testImplementation project(path: ':iceberg-parquet')
     testImplementation project(path: ':iceberg-hive-metastore')
     testImplementation project(path: ':iceberg-api', configuration: 'testArtifacts')
     testImplementation project(path: ':iceberg-core', configuration: 'testArtifacts')
-Original file line number
+Diff line change
@@ Expand Up / @@ -54,8 +54,6 @@ dependencies { @@
     }
     ```
-    You may also want to include `iceberg-parquet` for Parquet file support.
     ### Maven
     To add a dependency on Iceberg in Maven, add the following to your `pom.xml`:
@@ Expand Down @@