Skip to content

Comments

refactor(services/ipmfs): split ipmfs into separate crate#7042

Merged
koushiro merged 5 commits intoapache:mainfrom
tao12345666333:refactor/split-service-ipmfs
Dec 19, 2025
Merged

refactor(services/ipmfs): split ipmfs into separate crate#7042
koushiro merged 5 commits intoapache:mainfrom
tao12345666333:refactor/split-service-ipmfs

Conversation

@tao12345666333
Copy link
Member

@tao12345666333 tao12345666333 commented Dec 16, 2025

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

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

Closes #6903

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
This PR splits the ipmfs service into its own crate `opendal-service-ipmfs`.

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

Closes apache#6903
@tao12345666333 tao12345666333 force-pushed the refactor/split-service-ipmfs branch from 446126e to 860e676 Compare December 19, 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 19, 2025
- Remove http, huggingface, koofr, lakefs from core/core/src/services/mod.rs
- Remove their feature definitions from core/core/Cargo.toml
- These services are now separate crates in core/services/
@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

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 19, 2025
@koushiro koushiro merged commit 6eb8ec1 into apache:main Dec 19, 2025
336 of 337 checks passed
@tao12345666333 tao12345666333 deleted the refactor/split-service-ipmfs branch December 19, 2025 11:29
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: ipmfs

2 participants