Skip to content

[MINOR] Handle cancellation error with HoodieMetadataTableValidator#18371

Open
lokeshj1703 wants to merge 1 commit intoapache:masterfrom
lokeshj1703:minor-14
Open

[MINOR] Handle cancellation error with HoodieMetadataTableValidator#18371
lokeshj1703 wants to merge 1 commit intoapache:masterfrom
lokeshj1703:minor-14

Conversation

@lokeshj1703
Copy link
Copy Markdown
Collaborator

Describe the issue this Pull Request addresses

Currently cancellation error is not handled in HoodieMetadataTableValidator and all spark exceptions are thrown as validation errors. The PR fixes the checks so that cancellation errors are not thrown as validation exceptions.

Summary and Changelog

Currently cancellation error is not handled in HoodieMetadataTableValidator and all spark exceptions are thrown as validation errors. The PR fixes the checks so that cancellation errors are not thrown as validation exceptions.

Impact

NA

Risk Level

Low

Documentation Update

NA

Contributor's checklist

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

@github-actions github-actions bot added the size:M PR with lines of changes in (100, 300] label Mar 23, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.11%. Comparing base (817b3ad) to head (6406d88).

Files with missing lines Patch % Lines
...e/hudi/utilities/HoodieMetadataTableValidator.java 0.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master   #18371      +/-   ##
============================================
- Coverage     68.38%   66.11%   -2.28%     
+ Complexity    27505    22394    -5111     
============================================
  Files          2428     1994     -434     
  Lines        132926   110480   -22446     
  Branches      16000    13897    -2103     
============================================
- Hits          90902    73043   -17859     
+ Misses        34964    31238    -3726     
+ Partials       7060     6199     -861     
Flag Coverage Δ
common-and-other-modules ?
hadoop-mr-java-client 45.05% <0.00%> (-0.02%) ⬇️
spark-client-hadoop-common 48.21% <0.00%> (-0.01%) ⬇️
spark-java-tests 48.72% <80.00%> (+0.02%) ⬆️
spark-scala-tests 45.41% <0.00%> (+<0.01%) ⬆️
utilities 38.54% <40.00%> (-0.02%) ⬇️

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

Files with missing lines Coverage Δ
.../java/org/apache/hudi/exception/ExceptionUtil.java 100.00% <100.00%> (ø)
...e/hudi/utilities/HoodieMetadataTableValidator.java 65.24% <0.00%> (+0.07%) ⬆️

... and 776 files with indirect coverage changes

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

@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

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.

4 participants