Skip to content

[HUDI-8639] Support filter by partitions in ShowInvalidParquetProcedure#12410

Merged
zhangyue19921010 merged 2 commits intoapache:masterfrom
fhan688:support-filter-by-partitions-in-ShowInvalidParquetProcedure
Dec 5, 2024
Merged

[HUDI-8639] Support filter by partitions in ShowInvalidParquetProcedure#12410
zhangyue19921010 merged 2 commits intoapache:masterfrom
fhan688:support-filter-by-partitions-in-ShowInvalidParquetProcedure

Conversation

@fhan688
Copy link
Contributor

@fhan688 fhan688 commented Dec 3, 2024

Change Logs

add new parameter in ShowInvalidParquetProcedure, support specifying 'partitions' and filter invalid parquet files by partitions.

Impact

hudi-spark-datasource

Risk level (write none, low medium or high below)

Low

Documentation Update

None

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Dec 3, 2024
val javaRdd: JavaRDD[String] = jsc.parallelize(partitionPaths, partitionPaths.size())
val metadataConfig = HoodieMetadataConfig.newBuilder.enable(false).build
val metadata = HoodieTableMetadata.create(new HoodieSparkEngineContext(jsc), storage, metadataConfig, srcPath)
val partitionPaths: java.util.List[String] = metadata.getPartitionPathWithPathPrefixes(partitions.split(",").toList.asJava)
Copy link
Contributor

@zhangyue19921010 zhangyue19921010 Dec 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we use original FSUtils.getAllPartitionPaths( new HoodieSparkEngineContext(jsc), storage, srcPath, false) when users didn't set partitions config

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

getPartitionPathWithPathPrefixes(partitions) will return all partitionPaths when partitions are empty.

@github-actions github-actions bot added size:M PR with lines of changes in (100, 300] and removed size:S PR with lines of changes in (10, 100] labels Dec 4, 2024
@hudi-bot
Copy link
Collaborator

hudi-bot commented Dec 4, 2024

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

Copy link
Contributor

@zhangyue19921010 zhangyue19921010 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M PR with lines of changes in (100, 300]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants