[app-server] speed up thread list with lightweight SQLite rows#29355
[app-server] speed up thread list with lightweight SQLite rows#29355anaiskillian wants to merge 8 commits into
Conversation
|
@codex review |
|
Codex Review: Didn't find any major issues. What shall we delve into next? 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-list-sqlite-fast-path
branch
from
0336ac9 to
5bb8a00
Compare
|
@codex review |
![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: 5bb8a00b89
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
anaiskillian
force-pushed
the
anais/thread-list-sqlite-fast-path
branch
from
5bb8a00 to
1bba9a8
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1818f14c79
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
anaiskillian
force-pushed
the
anais/thread-list-sqlite-fast-path
branch
from
9d09931 to
c7d6ec8
Compare
anaiskillian
force-pushed
the
anais/thread-storage-metadata-foundation
branch
from
11cf96d to
7ddcf58
Compare
anaiskillian
force-pushed
the
anais/thread-list-sqlite-fast-path
branch
from
c7d6ec8 to
681edda
Compare
anaiskillian
force-pushed
the
anais/thread-storage-metadata-foundation
branch
from
3c3cc4b to
2d7edbc
Compare
1 participant
This pr routes local
thread/listrequests through a lightweight SQLite projection and batches filesystem scan repair work. It supersedes thethread/listportion of [app-server] improve thread list and resume rpc paths #28801 and is stacked on [thread store] separate thread names and repair ownership #29352.The fast path preserves filters, ordering, anchors, canonical parent IDs, and rollout existence validation for state-only and parent-filtered requests. Scan repair performs narrow location updates so stale row snapshots cannot overwrite concurrently updated thread metadata.