From 6d7b0dcd4d00352e784c04ec125c4d26c957b131 Mon Sep 17 00:00:00 2001 From: Timothy Brown Date: Wed, 8 Jan 2025 10:45:21 -0600 Subject: [PATCH] remove unused dependencies, start building bundles --- pom.xml | 11 ++ xtable-core/pom.xml | 37 +----- xtable-delta/pom.xml | 86 ++++++++++++-- xtable-hudi/pom.xml | 111 ++++++++++++++---- .../hudi/TestHudiFileStatsExtractor.java | 2 - xtable-iceberg/pom.xml | 101 ++++++++++------ xtable-integration-tests/pom.xml | 109 ++++++----------- 7 files changed, 289 insertions(+), 168 deletions(-) diff --git a/pom.xml b/pom.xml index 32143490..806f3768 100644 --- a/pom.xml +++ b/pom.xml @@ -154,6 +154,11 @@ parquet-avro ${parquet.version} + + org.apache.parquet + parquet-column + ${parquet.version} + @@ -322,6 +327,12 @@ ${spark.version} provided + + org.apache.spark + spark-catalyst_${scala.binary.version} + ${spark.version} + provided + commons-cli diff --git a/xtable-core/pom.xml b/xtable-core/pom.xml index 723ddeec..d06100dd 100644 --- a/xtable-core/pom.xml +++ b/xtable-core/pom.xml @@ -34,33 +34,6 @@ xtable-api ${project.version} - - org.apache.xtable - xtable-hudi-support-utils - ${project.version} - - - com.fasterxml.jackson.core - jackson-core - - - com.fasterxml.jackson.core - jackson-databind - - - com.fasterxml.jackson.module - jackson-module-scala_${scala.binary.version} - - - com.google.guava - guava - - - - - org.scala-lang - scala-library - @@ -74,10 +47,6 @@ org.apache.logging.log4j log4j-api - - org.apache.logging.log4j - log4j-1.2-api - @@ -114,6 +83,12 @@ log4j-slf4j2-impl test + + + com.google.guava + guava + test + diff --git a/xtable-delta/pom.xml b/xtable-delta/pom.xml index ba2b1552..0ec5dab0 100644 --- a/xtable-delta/pom.xml +++ b/xtable-delta/pom.xml @@ -50,18 +50,18 @@ jackson-databind - com.fasterxml.jackson.module - jackson-module-scala_${scala.binary.version} + com.fasterxml.jackson.core + jackson-annotations + com.google.guava guava - - org.apache.avro - avro + org.apache.commons + commons-lang3 @@ -93,10 +93,6 @@ org.apache.logging.log4j log4j-api - - org.apache.logging.log4j - log4j-1.2-api - @@ -104,6 +100,12 @@ spark-core_${scala.binary.version} provided + + org.apache.spark + spark-catalyst_${scala.binary.version} + ${spark.version} + provided + org.apache.spark spark-sql_${scala.binary.version} @@ -169,6 +171,72 @@ + + org.apache.maven.plugins + maven-shade-plugin + + + package + + shade + + + + + + + LICENSE + NOTICE + NOTICE.txt + + + + META-INF/LICENSE + target/classes/META-INF/LICENSE + + + META-INF/NOTICE + target/classes/META-INF/NOTICE + + + + + com.fasterxml.jackson.core:jackson-core + com.fasterxml.jackson.core:jackson-databind + com.fasterxml.jackson.core:jackson-annotations + org.scala-lang:scala-library + io.delta:delta-core_${scala.binary.version} + io.delta:delta-storage + org.apache.commons:commons-lang3 + com.google.guava:guava + + + + + com.fasterxml.jackson. + org.apache.xtable.shade.com.fasterxml.jackson. + + + io.delta. + org.apache.xtable.shade.io.delta. + + + org.apache.spark.sql.delta. + org.apache.xtable.shade.org.apache.spark.sql.delta. + + + org.apache.commons. + org.apache.xtable.shade.org.apache.commons. + + + com.google.guava. + org.apache.xtable.shade.com.google.guava. + + + + + + diff --git a/xtable-hudi/pom.xml b/xtable-hudi/pom.xml index 02233c87..ebc5d3e6 100644 --- a/xtable-hudi/pom.xml +++ b/xtable-hudi/pom.xml @@ -46,18 +46,7 @@ xtable-hudi-support-utils ${project.version} - - com.fasterxml.jackson.core - jackson-core - - - com.fasterxml.jackson.core - jackson-databind - - - com.fasterxml.jackson.module - jackson-module-scala_${scala.binary.version} - + com.google.guava guava @@ -69,19 +58,27 @@ avro + + + org.apache.parquet + parquet-column + + org.apache.hudi hudi-spark${spark.version.prefix}-bundle_${scala.binary.version} test - - org.apache.hudi - hudi-common - org.apache.hudi hudi-java-client + + + org.apache.hbase + hbase-server + + @@ -96,12 +93,8 @@ org.apache.logging.log4j log4j-api - - org.apache.logging.log4j - log4j-1.2-api - - + org.apache.spark spark-core_${scala.binary.version} @@ -149,6 +142,12 @@ test + + org.apache.commons + commons-lang3 + test + + org.apache.xtable xtable-core_${scala.binary.version} @@ -172,6 +171,76 @@ + + org.apache.maven.plugins + maven-shade-plugin + + + package + + shade + + + + + + + LICENSE + NOTICE + NOTICE.txt + + + + META-INF/LICENSE + target/classes/META-INF/LICENSE + + + META-INF/NOTICE + target/classes/META-INF/NOTICE + + + + + org.apache.xtable:xtable-hudi-support-utils + com.fasterxml.jackson.core:jackson-databind + com.fasterxml.jackson.datatype:jackson-datatype-jsr310 + com.fasterxml.jackson.core:jackson-core + org.apache.parquet:parquet-column + org.apache.parquet:parquet-common + org.apache.parquet:parquet-encoding + org.apache.hudi:hudi-java-client + org.apache.hudi:hudi-client-common + org.apache.hudi:hudi-common + org.apache.avro:avro + com.google.guava:guava + + + + + org.apache.parquet. + org.apache.xtable.shade.org.apache.parquet. + + + org.apache.hudi. + org.apache.xtable.shade.org.apache.hudi. + + + org.apache.avro. + org.apache.xtable.shade.org.apache.avro. + + + com.google.guava. + org.apache.xtable.shade.com.google.guava. + + + com.fasterxml.jackson. + org.apache.xtable.shade.com.fasterxml.jackson. + + + + + + diff --git a/xtable-hudi/src/test/java/org/apache/xtable/hudi/TestHudiFileStatsExtractor.java b/xtable-hudi/src/test/java/org/apache/xtable/hudi/TestHudiFileStatsExtractor.java index 4958a839..5b8ce32b 100644 --- a/xtable-hudi/src/test/java/org/apache/xtable/hudi/TestHudiFileStatsExtractor.java +++ b/xtable-hudi/src/test/java/org/apache/xtable/hudi/TestHudiFileStatsExtractor.java @@ -49,7 +49,6 @@ import org.apache.parquet.avro.AvroParquetWriter; import org.apache.parquet.hadoop.ParquetWriter; import org.apache.parquet.hadoop.util.HadoopOutputFile; -import org.jetbrains.annotations.NotNull; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.io.TempDir; @@ -447,7 +446,6 @@ private GenericRecord createRecord( return record; } - @NotNull private GenericData.Record getNestedRecord(Integer nestedIntValue) { GenericData.Record nested = new GenericData.Record(NESTED_SCHEMA); nested.put("nested_int", nestedIntValue); diff --git a/xtable-iceberg/pom.xml b/xtable-iceberg/pom.xml index 59a2dbe8..d741e1e0 100644 --- a/xtable-iceberg/pom.xml +++ b/xtable-iceberg/pom.xml @@ -42,23 +42,6 @@ provided - - com.fasterxml.jackson.core - jackson-core - - - com.fasterxml.jackson.core - jackson-databind - - - com.fasterxml.jackson.module - jackson-module-scala_${scala.binary.version} - - - com.google.guava - guava - - org.apache.avro @@ -75,38 +58,35 @@ iceberg-api - - - org.apache.hadoop - hadoop-common - provided - - org.apache.logging.log4j log4j-api + + - org.apache.logging.log4j - log4j-1.2-api + com.google.guava + guava + test - + + org.apache.hadoop + hadoop-common + provided + + + org.apache.iceberg iceberg-spark-runtime-${spark.version.prefix}_${scala.binary.version} test - org.apache.spark - spark-core_${scala.binary.version} - provided - - - org.apache.spark - spark-sql_${scala.binary.version} - provided + org.apache.hadoop + hadoop-client-runtime + test @@ -168,6 +148,57 @@ + + org.apache.maven.plugins + maven-shade-plugin + + + package + + shade + + + + + + + LICENSE + NOTICE + NOTICE.txt + + + + META-INF/LICENSE + target/classes/META-INF/LICENSE + + + META-INF/NOTICE + target/classes/META-INF/NOTICE + + + + + org.apache.iceberg:iceberg-core + org.apache.iceberg:iceberg-api + org.apache.iceberg:iceberg-common + org.apache.iceberg:iceberg-bundled-guava + org.apache.avro:avro + + + + + org.apache.iceberg. + org.apache.xtable.shade.org.apache.iceberg. + + + org.apache.avro. + org.apache.xtable.shade.org.apache.avro. + + + + + + diff --git a/xtable-integration-tests/pom.xml b/xtable-integration-tests/pom.xml index 3b1d66af..cf960b2e 100644 --- a/xtable-integration-tests/pom.xml +++ b/xtable-integration-tests/pom.xml @@ -29,63 +29,21 @@ XTable Project Integration Test Suite - - org.apache.xtable - xtable-api - ${project.version} - - - org.apache.xtable - xtable-core_${scala.binary.version} - ${project.version} - - - org.apache.xtable - xtable-delta_${scala.binary.version} - ${project.version} - - - org.apache.xtable - xtable-iceberg - ${project.version} - - - org.apache.xtable - xtable-hudi - ${project.version} - - - org.apache.xtable - xtable-hudi-support-utils - ${project.version} - com.fasterxml.jackson.core jackson-core + test com.fasterxml.jackson.core jackson-databind + test - - com.fasterxml.jackson.module - jackson-module-scala_${scala.binary.version} - + com.google.guava guava - - - - - org.apache.avro - avro - - - - - org.scala-lang - scala-library + test @@ -94,33 +52,15 @@ hudi-spark${spark.version.prefix}-bundle_${scala.binary.version} test - - org.apache.hudi - hudi-common - - - org.apache.hudi - hudi-java-client - - org.apache.iceberg iceberg-core + test org.apache.iceberg iceberg-api - - - - - io.delta - delta-core_${scala.binary.version} - - - io.delta - delta-standalone_${scala.binary.version} test @@ -150,18 +90,47 @@ org.apache.spark spark-core_${scala.binary.version} - provided + test org.apache.spark spark-sql_${scala.binary.version} - provided + test + + + org.apache.spark + spark-catalyst_${scala.binary.version} + test - - org.mockito - mockito-core + org.apache.xtable + xtable-api + ${project.version} + test + + + org.apache.xtable + xtable-core_${scala.binary.version} + ${project.version} + test + + + org.apache.xtable + xtable-delta_${scala.binary.version} + ${project.version} + test + + + org.apache.xtable + xtable-iceberg + ${project.version} + test + + + org.apache.xtable + xtable-hudi + ${project.version} test