kayg/calternal
Watch
1
0
Fork
You've already forked calternal
 0
Code Issues 191 Pull requests 61 Projects Releases Packages Wiki Activity Actions 20

Perf: search measured once its index has caught up #291

Closed
kayg wants to merge 0 commits from issue/44-perf-search-steady into dev
pull from: issue/44-perf-search-steady
merge into: kayg:dev
Conversation 1 Commits - Files changed -
kayg commented 2026-10-14 14:16:29 +00:00
Owner
Copy link

Part of #44. Harness only.

search.query ran while a first search build (2 to 4 minutes at 100k notes on the perf VM) was still going, so one run's build decided the p95 and runs disagreed by over 100%. Now: the first query is timed on its own (search.first_query, target p95 <= 1 s, not gated), the harness waits for the index to catch up (search.catch_up), then measures search.query (gate p95 <= 100 ms) as people see it every day.

🤖 Generated with Claude Code

Part of #44. Harness only. `search.query` ran while a first search build (2 to 4 minutes at 100k notes on the perf VM) was still going, so one run's build decided the p95 and runs disagreed by over 100%. Now: the first query is timed on its own (`search.first_query`, target p95 <= 1 s, not gated), the harness waits for the index to catch up (`search.catch_up`), then measures `search.query` (gate p95 <= 100 ms) as people see it every day. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Perf: search measured once its index has caught up
Some checks failed
CI / server (pull_request) Successful in 21s
Details
CI / web (pull_request) Failing after 21s
Details
CI / rust (pull_request) Successful in 1m33s
Details
34af2271df 
`search.query` ran while a first search build (2 to 4 minutes at 100,000
notes on the perf VM) was still going, so one run's build decided the
p95 and the spread between runs was over 100%. The harness now times the
first query on its own (`search.first_query`), waits for the index to
catch up (`search.catch_up`), then measures `search.query`.

Part of #44

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018Yyy8fQGxPLkLa8v8NXz7T
kayg commented 2026-10-15 02:22:48 +00:00
Author
Owner
Copy link

Merged into dev via train j (#391).

Merged into dev via train j (#391).
kayg closed this pull request 2026-10-15 02:22:50 +00:00
Some checks failed
CI / server (pull_request) Successful in 21s
Details
CI / web (pull_request) Failing after 21s
Details
CI / rust (pull_request) Successful in 1m33s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
backend
Rust server

  
design
Mockups, UI and TASTE

  
docs
ADRs, CONTEXT, TASTE

  
idea
Parked idea, never worked until promoted

  
infra
Deploy, CI, VMs

  
perf
Performance and budgets

  
reliability
Data integrity, corruption traps

  
web
Web client

No labels
backend
design
docs
idea
infra
perf
reliability
web
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
kayg/calternal!291
No description provided.