Skip to content

DNM: testing index refactor#18364

Closed
cshuo wants to merge 1 commit into
apache:masterfrom
cshuo:indexer_abstraction_v2
Closed

DNM: testing index refactor#18364
cshuo wants to merge 1 commit into
apache:masterfrom
cshuo:indexer_abstraction_v2

Conversation

@cshuo
Copy link
Copy Markdown
Collaborator

@cshuo cshuo commented Mar 21, 2026

Describe the issue this Pull Request addresses

Summary and Changelog

Impact

Risk Level

Documentation Update

Contributor's checklist

  • Read through contributor's guide
  • Enough context is provided in the sections above
  • Adequate tests were added if applicable

@cshuo cshuo marked this pull request as draft March 21, 2026 07:06
@github-actions github-actions Bot added the size:XL PR with lines of changes > 1000 label Mar 21, 2026
@cshuo cshuo force-pushed the indexer_abstraction_v2 branch from 0a78a60 to 88b080a Compare March 21, 2026 10:00
@cshuo cshuo force-pushed the indexer_abstraction_v2 branch from 88b080a to 87ebeba Compare March 21, 2026 10:13
@hudi-bot
Copy link
Copy Markdown
Collaborator

CI report:

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

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.16667% with 52 lines in your changes missing coverage. Please review.
✅ Project coverage is 68.48%. Comparing base (14a549f) to head (87ebeba).
⚠️ Report is 10 commits behind head on master.

Files with missing lines Patch % Lines
.../hudi/metadata/index/record/BaseRecordIndexer.java 71.84% 25 Missing and 4 partials ⚠️
...dex/UnsupportedExpressionIndexRecordGenerator.java 42.85% 4 Missing ⚠️
...i/metadata/index/expression/ExpressionIndexer.java 88.23% 1 Missing and 3 partials ⚠️
...hudi/metadata/HoodieBackedTableMetadataWriter.java 95.23% 1 Missing and 2 partials ⚠️
...org/apache/hudi/metadata/index/IndexerFactory.java 85.71% 2 Missing and 1 partial ⚠️
...ata/index/SparkExpressionIndexRecordGenerator.java 80.00% 2 Missing and 1 partial ⚠️
...metadata/index/columnstats/ColumnStatsIndexer.java 94.28% 1 Missing and 1 partial ⚠️
...udi/metadata/index/secondary/SecondaryIndexer.java 90.47% 0 Missing and 2 partials ⚠️
...apache/hudi/metadata/index/files/FilesIndexer.java 95.00% 0 Missing and 1 partial ⚠️
...etadata/index/record/PartitionedRecordIndexer.java 96.42% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18364      +/-   ##
============================================
- Coverage     68.48%   68.48%   -0.01%     
- Complexity    27362    27539     +177     
============================================
  Files          2420     2445      +25     
  Lines        132127   132732     +605     
  Branches      15909    15990      +81     
============================================
+ Hits          90491    90897     +406     
- Misses        34627    34788     +161     
- Partials       7009     7047      +38     
Flag Coverage Δ
common-and-other-modules 44.56% <63.33%> (+0.18%) ⬆️
hadoop-mr-java-client 45.12% <57.55%> (+<0.01%) ⬆️
spark-client-hadoop-common 48.22% <47.16%> (-0.12%) ⬇️
spark-java-tests 48.82% <74.89%> (-0.10%) ⬇️
spark-scala-tests 44.96% <71.96%> (-0.16%) ⬇️
utilities 38.67% <70.71%> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...oodieBackedTableMetadataWriterTableVersionSix.java 76.41% <100.00%> (+0.73%) ⬆️
...apache/hudi/metadata/HoodieMetadataWriteUtils.java 83.49% <100.00%> (+0.90%) ⬆️
.../metadata/SecondaryIndexRecordGenerationUtils.java 88.27% <100.00%> (ø)
...va/org/apache/hudi/metadata/index/BaseIndexer.java 100.00% <100.00%> (ø)
...tadata/index/bloomfilters/BloomFiltersIndexer.java 100.00% <100.00%> (ø)
...data/index/model/IndexPartitionInitialization.java 100.00% <100.00%> (ø)
...ta/index/partitionstats/PartitionStatsIndexer.java 100.00% <100.00%> (ø)
...ache/hudi/metadata/index/record/RecordIndexer.java 100.00% <100.00%> (ø)
...metadata/FlinkHoodieBackedTableMetadataWriter.java 85.71% <100.00%> (-0.34%) ⬇️
.../metadata/JavaHoodieBackedTableMetadataWriter.java 85.18% <100.00%> (-0.53%) ⬇️
... and 20 more

... and 48 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cshuo cshuo closed this Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL PR with lines of changes > 1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants