Skip to content

Comments

refactor: split gridfs service out as new crate#7037

Merged
koushiro merged 6 commits intoapache:mainfrom
codxbrexx:issue-A
Dec 20, 2025
Merged

refactor: split gridfs service out as new crate#7037
koushiro merged 6 commits intoapache:mainfrom
codxbrexx:issue-A

Conversation

@codxbrexx
Copy link
Contributor

@codxbrexx codxbrexx commented Dec 16, 2025

Which issue does this PR close?

Part of #6829
Closes #6898

Rationale for this change

This PR extracts the gridfs service from opendal-core into its own crate opendal-service-gridfs as part of the core-split effort (RFC-6828). This helps separate service-specific dependencies (like mongodb which is heavy) from the core logic, improving maintainability and compilation speed for users who don't need this service.

What changes are included in this PR?

  • Created a new crate core/services/gridfs.
  • Moved implementation files (backend.rs, core.rs, writer.rs, etc.) from core/core/src/services/gridfs to the new crate.
  • Removed gridfs logic and dependencies from opendal-core.
  • Updated feature flags: services-gridfs in opendal now points to opendal-service-gridfs.
  • Updated lib.rs in the facade to re-export GridFs.

Are there any user-facing changes?

No. The public API opendal::services::GridFs remains unchanged. Users can continue to use the service exactly as before, provided they have the services-gridfs feature enabled.

AI Usage Statement

AI tools were used only for understanding existing code and project documentation.
All code changes were written and reviewed manually.

@codxbrexx codxbrexx requested a review from Xuanwo as a code owner December 16, 2025 21:24
@dosubot dosubot bot added size:L This PR changes 100-499 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
@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 16, 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 16, 2025
@Xuanwo Xuanwo changed the title Service: gridfs #6898 refactor: split gridfs service out as new crate Dec 17, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 20, 2025
@koushiro koushiro merged commit 5abf36a into apache:main Dec 20, 2025
591 of 592 checks passed
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: gridfs

3 participants