Skip to content

[trees] Perf | Fast-path simple segment sort-key comparisons#493

Merged
SlexAxton merged 1 commit intomainfrom
autoresearch/pathstore-visible-rows-ready/02-segment-sort-fast-path
Apr 10, 2026
Merged

[trees] Perf | Fast-path simple segment sort-key comparisons#493
SlexAxton merged 1 commit intomainfrom
autoresearch/pathstore-visible-rows-ready/02-segment-sort-fast-path

Conversation

@SlexAxton
Copy link
Copy Markdown
Contributor

Teach compareSegmentSortKeys() to compare the common single-string, no-numeric segment keys directly on their cached lowercase values instead of walking token arrays. This trims remaining initializeExpandedPaths comparison overhead.

Experiments: #31
Metric: visible_rows_ready_ms 197.5 → 192.9 (-2.3%)

Teach compareSegmentSortKeys() to compare the common single-string, no-numeric segment keys directly on their cached lowercase values instead of walking token arrays. This trims remaining initializeExpandedPaths comparison overhead.

Experiments: #31
Metric: visible_rows_ready_ms 197.5 → 192.9 (-2.3%)
@vercel
Copy link
Copy Markdown

vercel bot commented Apr 10, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
pierrejs-diff-demo Ready Ready Preview Apr 10, 2026 2:50am
pierrejs-docs Ready Ready Preview Apr 10, 2026 2:50am

Request Review

@SlexAxton SlexAxton merged commit 911437f into main Apr 10, 2026
10 checks passed
@SlexAxton SlexAxton deleted the autoresearch/pathstore-visible-rows-ready/02-segment-sort-fast-path branch April 10, 2026 02:54
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