issue-1541: drain HandleOpsQueue when AsyncDestroy is disabled#6562
issue-1541: drain HandleOpsQueue when AsyncDestroy is disabled#6562WilyTiger wants to merge 1 commit into
Conversation
WilyTiger
added
large-tests
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo.
|
93c73ad to
f603b15
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 93c73adcda
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
Note This is an automated comment that will be appended during run. Note All workloads for linux-x86_64-relwithdebinfo have completed. Tip Planned checks for linux-x86_64-relwithdebinfo. 🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 7201s): all tests PASSED for commit f603b15.
|
| FileSystemConfig->GetAsyncDestroyHandleEnabled() || | ||
| FileSystemConfig->GetAsyncDestroyReadOnlyHandleEnabled()) |
There was a problem hiding this comment.
Let's extract this into a variable and reuse it on lines 1024-1025 as well?
2 participants
Notes
HandleOpsQueue is now restored and drained when AsyncDestroy is disabled if a persistent queue from a previous session exists.
New Release requests continue to use the synchronous path while previously queued DestroyHandle requests are processed in the background.
Issue
#1541