[app-server] speed up thread resume without deferred repair#29357
[app-server] speed up thread resume without deferred repair#29357anaiskillian wants to merge 5 commits into
Conversation
|
@codex review |
|
Codex Review: Didn't find any major issues. 🚀 Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
anaiskillian
force-pushed
the
anais/thread-resume-history-fast-path
branch
from
c655f37 to
9113fa5
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
anaiskillian
force-pushed
the
anais/thread-resume-history-fast-path
branch
from
9113fa5 to
39f9868
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Swish! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
anaiskillian
force-pushed
the
anais/thread-resume-history-fast-path
branch
from
39f9868 to
8f7f8a0
Compare
anaiskillian
force-pushed
the
anais/thread-storage-metadata-foundation
branch
from
11cf96d to
7ddcf58
Compare
anaiskillian
force-pushed
the
anais/thread-resume-history-fast-path
branch
from
8f7f8a0 to
dd77633
Compare
anaiskillian
force-pushed
the
anais/thread-storage-metadata-foundation
branch
from
3c3cc4b to
2d7edbc
Compare
1 participant
This pr speeds up local
thread/resumeby parsing plain rollout files on a blocking worker, reusing loaded history, and avoiding duplicate history clones and reads. It supersedes thethread/resumeportion of [app-server] improve thread list and resume rpc paths #28801 and is stacked on [thread store] separate thread names and repair ownership #29352.Resume metadata is derived from the already-loaded history before persistence opens, so a later append cannot replay stale history metadata after an explicit rename. Cold, running, metadata-only, token-usage, and rename-then-turn resume coverage preserves the existing RPC behavior.