Skip to content

snapshot: Don't fsync directories on Windows#4939

Open
kim wants to merge 1 commit intomasterfrom
kim/snapshot/win-no-fsync-dir
Open

snapshot: Don't fsync directories on Windows#4939
kim wants to merge 1 commit intomasterfrom
kim/snapshot/win-no-fsync-dir

Conversation

@kim
Copy link
Copy Markdown
Contributor

@kim kim commented May 4, 2026

One shall not fsync a directory on Windows, or else receive an unhelpful access denied error.

Caught by the Windows smoketests for #4892 (failure to create a snapshot wouldn't fail the smoketests, perhaps they should in the future).

One shall not fsync a directory on Windows, or else receive an unhelpful
access denied error.

Caught by the Windows smoketests for #4892 (failure to create a snapshot
wouldn't fail the smoketests, perhaps they should in the future).
@kim kim requested a review from gefjon May 4, 2026 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant