Skip to content

RFC-6678: Simulate Layer#6678

Merged
Xuanwo merged 3 commits intomainfrom
6676-start-after-on-fs
Oct 19, 2025
Merged

RFC-6678: Simulate Layer#6678
Xuanwo merged 3 commits intomainfrom
6676-start-after-on-fs

Conversation

@Xuanwo
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo commented Oct 16, 2025

Which issue does this PR close?

None.

Rationale for this change

This RFC will make OpenDAL's internal simulate logic cleaner and more controllable. For example, users like scopedb (cc @tisonkun and @leiysky) who don't care about stat_dir or create_dir can disable the simulate logic entirely.

This change will also make is easier to add new simulate logic.

What changes are included in this PR?

A new RFC.

Are there any user-facing changes?

This PR was primarily authored with Amp using Claude-Sonnet-4.5 and then hand-reviewed by me. I AM responsible for every change made in this PR. I aimed to keep it aligned with our goals, though I may have missed minor issues. Please flag anything that feels off, I'll fix it quickly.

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo changed the title RFC: Simulate Layer RFC-6678: Simulate Layer Oct 16, 2025
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Oct 16, 2025
Signed-off-by: Xuanwo <github@xuanwo.io>
Signed-off-by: Xuanwo <github@xuanwo.io>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 19, 2025
@Xuanwo Xuanwo merged commit 68b4129 into main Oct 19, 2025
80 checks passed
@Xuanwo Xuanwo deleted the 6676-start-after-on-fs branch October 19, 2025 13:06
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 size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants