Skip to content

Comments

refactor(layers/dtrace): split dtrace layer into separate crate#7078

Merged
koushiro merged 1 commit intoapache:mainfrom
PiyushXCoder:dtrace
Dec 21, 2025
Merged

refactor(layers/dtrace): split dtrace layer into separate crate#7078
koushiro merged 1 commit intoapache:mainfrom
PiyushXCoder:dtrace

Conversation

@PiyushXCoder
Copy link
Contributor

Which issue does this PR close?

Closes #6935

Rationale for this change

Relates to #6829

What changes are included in this PR?

This PR extracts the dtrace layer implementation from opendal-core into a standalone crate opendal-layer-dtrace, while keeping the public surface opendal::layers::DtraceLayer unchanged.

  • Keeps feature flag layers-dtrace intact.
  • Moves dtrace-specific dependency (probe) to the new layer crate.
  • Removed the old in-tree dtrace layer module and its feature/deps from opendal-core.

Are there any user-facing changes?

No user-facing changes. The public API opendal::layers::DtraceLayer remains unchanged.

AI Usage Statement

No AI has been used

@PiyushXCoder PiyushXCoder requested a review from Xuanwo as a code owner December 20, 2025 17:35
@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 20, 2025
@PiyushXCoder
Copy link
Contributor Author

Thank you for the review. I have made the changes. I didn't knew dtrace is available only on Linux.

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.

Thanks for your contribution

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Dec 21, 2025
@koushiro koushiro merged commit 9ffbad7 into apache:main Dec 21, 2025
335 checks passed
@PiyushXCoder PiyushXCoder deleted the dtrace branch December 27, 2025 19:23
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:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[core split] Layer: dtrace

2 participants