unical1988 commented on code in PR #728: URL: https://github.com/apache/incubator-xtable/pull/728#discussion_r2331164482
########## xtable-core/src/test/java/org/apache/xtable/parquet/ITParquetConversionSource.java: ########## @@ -0,0 +1,435 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one + * or more contributor license agreements. See the NOTICE file + * distributed with this work for additional information + * regarding copyright ownership. The ASF licenses this file + * to you under the Apache License, Version 2.0 (the + * "License"); you may not use this file except in compliance + * with the License. You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.apache.xtable.parquet; + +import static org.apache.xtable.GenericTable.getTableName; +import static org.apache.xtable.model.storage.TableFormat.*; +import static org.junit.jupiter.api.Assertions.assertEquals; +import static org.junit.jupiter.api.Assertions.assertFalse; + +import java.net.URISyntaxException; +import java.nio.file.Path; +import java.sql.Timestamp; +import java.time.Duration; +import java.time.Instant; +import java.time.ZoneId; +import java.time.format.DateTimeFormatter; +import java.time.temporal.ChronoUnit; +import java.util.*; +import java.util.Arrays; +import java.util.Collections; +import java.util.HashMap; +import java.util.List; +import java.util.Map; +import java.util.Optional; +import java.util.Properties; +import java.util.function.Function; +import java.util.stream.Collectors; +import java.util.stream.Stream; + +import lombok.Builder; +import lombok.Value; + +import org.apache.spark.SparkConf; +import org.apache.spark.api.java.JavaSparkContext; +import org.apache.spark.sql.*; +import org.apache.spark.sql.types.DataTypes; +import org.apache.spark.sql.types.MetadataBuilder; +import org.apache.spark.sql.types.StructField; +import org.apache.spark.sql.types.StructType; +import org.junit.jupiter.api.AfterAll; +import org.junit.jupiter.api.BeforeAll; +import org.junit.jupiter.api.io.TempDir; +import org.junit.jupiter.params.ParameterizedTest; +import org.junit.jupiter.params.provider.Arguments; +import org.junit.jupiter.params.provider.MethodSource; + +import org.apache.hudi.client.HoodieReadClient; +import org.apache.hudi.common.config.HoodieMetadataConfig; + +import com.fasterxml.jackson.databind.ObjectMapper; + +import org.apache.xtable.GenericTable; +import org.apache.xtable.conversion.*; +import org.apache.xtable.conversion.ConversionConfig; +import org.apache.xtable.conversion.ConversionController; +import org.apache.xtable.conversion.ConversionSourceProvider; +import org.apache.xtable.conversion.SourceTable; +import org.apache.xtable.conversion.TargetTable; +import org.apache.xtable.hudi.HudiTestUtil; +import org.apache.xtable.model.sync.SyncMode; + +public class ITParquetConversionSource { + private static final DateTimeFormatter DATE_FORMAT = + DateTimeFormatter.ofPattern("yyyy-MM-dd HH:mm:ss.SSS").withZone(ZoneId.of("UTC")); + private static final ObjectMapper OBJECT_MAPPER = new ObjectMapper(); + public static final String PARTITION_FIELD_SPEC_CONFIG = + "xtable.parquet.source.partition_field_spec_config"; + @TempDir public static Path tempDir; + private static JavaSparkContext jsc; + private static SparkSession sparkSession; + private static StructType schema; + + @BeforeAll + public static void setupOnce() { + SparkConf sparkConf = HudiTestUtil.getSparkConf(tempDir); + + String extraJavaOptions = "--add-opens=java.base/sun.nio.ch=ALL-UNNAMED"; + sparkConf.set("spark.driver.extraJavaOptions", extraJavaOptions); + sparkConf = HoodieReadClient.addHoodieSupport(sparkConf); + sparkConf.set("parquet.avro.write-old-list-structure", "false"); + + String javaOpts = + "--add-opens=java.base/java.nio=ALL-UNNAMED " + + "--add-opens=java.base/java.lang=ALL-UNNAMED " + + "--add-opens=java.base/java.util=ALL-UNNAMED " + + "--add-opens=java.base/java.util.concurrent=ALL-UNNAMED " + + "--add-opens=java.base/java.io=ALL-UNNAMED" + + "--add-opens=java.base/java.lang.invoke=ALL-UNNAMED" + + "--add-opens=java.base/java.lang.invoke=ALL-UNNAMED"; + + sparkConf.set("spark.driver.extraJavaOptions", javaOpts); + sparkConf.set("spark.executor.extraJavaOptions", javaOpts); + + sparkSession = SparkSession.builder().config(sparkConf).getOrCreate(); + jsc = JavaSparkContext.fromSparkContext(sparkSession.sparkContext()); + + List<Row> data = + Arrays.asList( + RowFactory.create(1, "Alice", 30, new Timestamp(System.currentTimeMillis())), + RowFactory.create(2, "Bob", 24, new Timestamp(System.currentTimeMillis() + 1000)), + RowFactory.create(3, "Charlie", 35, new Timestamp(System.currentTimeMillis() + 2000)), + RowFactory.create(4, "David", 29, new Timestamp(System.currentTimeMillis() + 3000)), + RowFactory.create(5, "Eve", 22, new Timestamp(System.currentTimeMillis() + 4000))); + + schema = + DataTypes.createStructType( + new StructField[] { + DataTypes.createStructField("id", DataTypes.IntegerType, false), + DataTypes.createStructField("name", DataTypes.StringType, false), + DataTypes.createStructField("age", DataTypes.IntegerType, false), + DataTypes.createStructField( + "timestamp", + DataTypes.TimestampType, + false, + new MetadataBuilder().putString("precision", "millis").build()) + }); + + Dataset<Row> df = sparkSession.createDataFrame(data, schema); + df.withColumn("year", functions.year(functions.col("timestamp").cast(DataTypes.TimestampType))) + .write() + .mode(SaveMode.Overwrite) + .partitionBy("year") + .parquet(tempDir.toAbsolutePath().toString()); + + // test if data was written correctly + Dataset<Row> reloadedDf = sparkSession.read().parquet(tempDir.toAbsolutePath().toString()); + reloadedDf.show(); + reloadedDf.printSchema(); + } + + @AfterAll + public static void teardown() { + if (jsc != null) { + jsc.stop(); + jsc = null; + } + if (sparkSession != null) { + sparkSession.stop(); + sparkSession = null; + } + } + + private static Stream<Arguments> provideArgsForFilePartitionTesting() { + String timestampFilter = + String.format( + "timestamp_micros_nullable_field < timestamp_millis(%s)", + Instant.now().truncatedTo(ChronoUnit.DAYS).minus(2, ChronoUnit.DAYS).toEpochMilli()); + String levelFilter = "level = 'INFO'"; + String nestedLevelFilter = "nested_record.level = 'INFO'"; + String severityFilter = "severity = 1"; + String timestampAndLevelFilter = String.format("%s and %s", timestampFilter, levelFilter); + return Stream.of( + Arguments.of( + buildArgsForPartition( + PARQUET, + Arrays.asList(ICEBERG, DELTA, HUDI), + "timestamp:YEAR:year=YYYY", // ts:DAY:year=YYYY/month=MM/day=DD + // "year=YYYY/month=MM/day=DD" + "timestamp:YEAR:year=YYYY", + levelFilter))); + } + + private static TableFormatPartitionDataHolder buildArgsForPartition( + String sourceFormat, + List<String> targetFormats, + String hudiPartitionConfig, + String xTablePartitionConfig, + String filter) { + return TableFormatPartitionDataHolder.builder() + .sourceTableFormat(sourceFormat) + .targetTableFormats(targetFormats) + .hudiSourceConfig(Optional.ofNullable(hudiPartitionConfig)) + .xTablePartitionConfig(xTablePartitionConfig) + .filter(filter) + .build(); + } + + private static ConversionConfig getTableSyncConfig( + String sourceTableFormat, + SyncMode syncMode, + String tableName, + GenericTable table, + List<String> targetTableFormats, + String partitionConfig, + Duration metadataRetention) { + Properties sourceProperties = new Properties(); + if (partitionConfig != null) { + sourceProperties.put(PARTITION_FIELD_SPEC_CONFIG, partitionConfig); + } + SourceTable sourceTable = + SourceTable.builder() + .name(tableName) + .formatName(sourceTableFormat) + .basePath(table.getBasePath()) + .dataPath(table.getDataPath()) + .additionalProperties(sourceProperties) + .build(); + + List<TargetTable> targetTables = + targetTableFormats.stream() + .map( + formatName -> + TargetTable.builder() + .name(tableName) + .formatName(formatName) + // set the metadata path to the data path as the default (required by Hudi) + .basePath(table.getDataPath()) + .metadataRetention(metadataRetention) + .build()) + .collect(Collectors.toList()); + + return ConversionConfig.builder() + .sourceTable(sourceTable) + .targetTables(targetTables) + .syncMode(syncMode) + .build(); + } + + private ConversionSourceProvider<?> getConversionSourceProvider(String sourceTableFormat) { + if (sourceTableFormat.equalsIgnoreCase(PARQUET)) { + ConversionSourceProvider<Long> parquetConversionSourceProvider = + new ParquetConversionSourceProvider(); + parquetConversionSourceProvider.init(jsc.hadoopConfiguration()); + return parquetConversionSourceProvider; + } else { + throw new IllegalArgumentException("Unsupported source format: " + sourceTableFormat); + } + } + + @ParameterizedTest + @MethodSource("provideArgsForFilePartitionTesting") + public void testFilePartitionedData( + TableFormatPartitionDataHolder tableFormatPartitionDataHolder) { + String tableName = getTableName(); + String sourceTableFormat = tableFormatPartitionDataHolder.getSourceTableFormat(); + List<String> targetTableFormats = tableFormatPartitionDataHolder.getTargetTableFormats(); + // Optional<String> hudiPartitionConfig = tableFormatPartitionDataHolder.getHudiSourceConfig(); + String xTablePartitionConfig = tableFormatPartitionDataHolder.getXTablePartitionConfig(); + String filter = tableFormatPartitionDataHolder.getFilter(); + ConversionSourceProvider<?> conversionSourceProvider = + getConversionSourceProvider(sourceTableFormat); + GenericTable table; + table = + GenericTable.getInstance(tableName, tempDir, sparkSession, jsc, sourceTableFormat, true); + try (GenericTable tableToClose = table) { + ConversionConfig conversionConfig = + getTableSyncConfig( + sourceTableFormat, + SyncMode.FULL, + tableName, + table, + targetTableFormats, + xTablePartitionConfig, + null); + ConversionController conversionController = + new ConversionController(jsc.hadoopConfiguration()); + conversionController.sync(conversionConfig, conversionSourceProvider); Review Comment: I will get into that at last. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@xtable.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
