Skip to content

fix: Avoid using HoodieHadoopStorage directly#17560

Merged
yihua merged 2 commits intoapache:masterfrom
CTTY:ctty/dehadoop-storage
Dec 12, 2025
Merged

fix: Avoid using HoodieHadoopStorage directly#17560
yihua merged 2 commits intoapache:masterfrom
CTTY:ctty/dehadoop-storage

Conversation

@CTTY
Copy link
Contributor

@CTTY CTTY commented Dec 11, 2025

Describe the issue this Pull Request addresses

Use StorageUtils.getStorage rather than using constructors directly

Summary and Changelog

  • Use StorageUtils.getStorage rather than using HoodieHadoopStorage constructor directly in multiple places

Impact

None

Risk Level

none

Documentation Update

none

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:S PR with lines of changes in (10, 100] label Dec 11, 2025
Copy link
Contributor

@yihua yihua left a comment

Choose a reason for hiding this comment

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

LGTM

@hudi-bot
Copy link
Collaborator

CI report:

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

@yihua yihua merged commit 96d86b6 into apache:master Dec 12, 2025
134 of 137 checks passed
@CTTY CTTY deleted the ctty/dehadoop-storage branch December 13, 2025 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S PR with lines of changes in (10, 100]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants