Skip to content

Conversation

@tao12345666333
Copy link
Member

@tao12345666333 tao12345666333 commented Dec 16, 2025

This PR splits the hdfs service into its own crate opendal-service-hdfs.

  • Create new crate at core/services/hdfs
  • Update feature flag to use the new crate
  • Remove hdfs from opendal-core
  • Re-export from facade crate

Closes #6899

AI Usage Statement

Claude Code + Claude 4.5 Opus

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" labels Dec 16, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Mostly LGTM, the only thing left is resovle the conflicts.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 17, 2025
@tao12345666333 tao12345666333 force-pushed the refactor/split-service-hdfs branch from 83b7f57 to cbdd599 Compare December 17, 2025 08:45
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Dec 17, 2025
@tao12345666333 tao12345666333 force-pushed the refactor/split-service-hdfs branch from de9470d to ab64a1a Compare December 17, 2025 13:28
@tao12345666333 tao12345666333 force-pushed the refactor/split-service-hdfs branch 2 times, most recently from b6e3f51 to 87ae517 Compare December 18, 2025 14:33
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Dec 18, 2025
This PR splits the hdfs service into its own crate `opendal-service-hdfs`.

- Create new crate at `core/services/hdfs`
- Update feature flag to use the new crate
- Remove hdfs from `opendal-core`
- Re-export from facade crate

Closes apache#6899
@tao12345666333 tao12345666333 force-pushed the refactor/split-service-hdfs branch from 87ae517 to 7fa3dae Compare December 19, 2025 11:42
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Dec 19, 2025
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:L This PR changes 100-499 lines, ignoring generated files. labels Dec 19, 2025
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Dec 19, 2025
Copy link
Member

@koushiro koushiro left a comment

Choose a reason for hiding this comment

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

Thank you

@koushiro koushiro merged commit 70ff36c into apache:main Dec 19, 2025
335 checks passed
@tao12345666333 tao12345666333 deleted the refactor/split-service-hdfs branch December 19, 2025 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[core split] Service: hdfs

3 participants