Skip to content

perf: find TokenTable index once#425

Open
dkharms wants to merge 1 commit into
mainfrom
0-token-table-scan
Open

perf: find TokenTable index once#425
dkharms wants to merge 1 commit into
mainfrom
0-token-table-scan

Conversation

@dkharms
Copy link
Copy Markdown
Member

@dkharms dkharms commented May 14, 2026

Description

TableLoader.Load previously re-scanned all token blocks on every call to find the start of the token table. This commit extracts that scan into advanceToTable, guarded by a sync.Once, so the linear scan over token blocks happens at most once per TableLoader instance.


  • I have read and followed all requirements in CONTRIBUTING.md;
  • I used LLM/AI assistance to make this pull request;

If you have used LLM/AI assistance please provide model name and full prompt:

Model: {{model-name}}
Prompt: {{prompt}}

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.63%. Comparing base (e0cf437) to head (bc14b51).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #425      +/-   ##
==========================================
- Coverage   70.65%   70.63%   -0.03%     
==========================================
  Files         219      219              
  Lines       16967    16968       +1     
==========================================
- Hits        11988    11985       -3     
- Misses       4084     4086       +2     
- Partials      895      897       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@dkharms dkharms added the performance Features or improvements that positively affect seq-db performance label May 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Features or improvements that positively affect seq-db performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants