Skip to content

feat: add token block size config - #439

Merged
cheb0 merged 1 commit into
mainfrom
0-token-block-size-config
Jul 27, 2026
Merged

feat: add token block size config#439
cheb0 merged 1 commit into
mainfrom
0-token-block-size-config

Conversation

@cheb0

@cheb0 cheb0 commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

Description

Not for performance or compression reasons, but mostly for tests. Smaller blocks can catch more edge cases.


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

@eguguchkin
eguguchkin requested review from eguguchkin and forshev June 15, 2026 09:17
@eguguchkin eguguchkin added this to the v0.73.0 milestone Jun 22, 2026
@dkharms
dkharms force-pushed the 336-merge-source branch from 0f1955e to aa2aecf Compare June 24, 2026 10:45
@eguguchkin eguguchkin modified the milestones: v0.73.0, v0.74.0 Jun 24, 2026
@dkharms
dkharms force-pushed the 336-merge-source branch 2 times, most recently from 50256a3 to b017111 Compare June 26, 2026 08:54
@dkharms
dkharms force-pushed the 336-merge-source branch 3 times, most recently from 8917131 to 261846d Compare July 6, 2026 12:37
Base automatically changed from 336-merge-source to main July 6, 2026 12:41
@dkharms dkharms modified the milestones: v0.74.0, v0.75.0 Jul 8, 2026
@cheb0
cheb0 force-pushed the 0-token-block-size-config branch 2 times, most recently from ec2d8f4 to 862caf8 Compare July 8, 2026 07:59
@codecov-commenter

codecov-commenter commented Jul 8, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.30769% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 70.35%. Comparing base (a79eceb) to head (7ea2f82).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
cmd/seq-db/seq-db.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #439      +/-   ##
==========================================
- Coverage   70.53%   70.35%   -0.18%     
==========================================
  Files         232      232              
  Lines       18319    18328       +9     
==========================================
- Hits        12921    12895      -26     
- Misses       4420     4464      +44     
+ Partials      978      969       -9     

☔ View full report in Codecov by Harness.
📢 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.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

🔴 Performance Degradation

Some benchmarks have degraded compared to the previous run.
Click on Show table button to see full list of degraded benchmarks.

Show table
Name Previous Current Ratio Verdict
MutexListAppend-4 25381c 8f3748
199.21 MB/s 179.00 MB/s 0.90 🔴

@cheb0
cheb0 force-pushed the 0-token-block-size-config branch from 862caf8 to 7ea2f82 Compare July 27, 2026 06:13
@cheb0
cheb0 enabled auto-merge (squash) July 27, 2026 06:16
@cheb0
cheb0 merged commit 4824281 into main Jul 27, 2026
8 checks passed
@cheb0
cheb0 deleted the 0-token-block-size-config branch July 27, 2026 06:16
@github-actions

Copy link
Copy Markdown
Contributor

🔴 Performance Degradation

Some benchmarks have degraded compared to the previous run.
Click on Show table button to see full list of degraded benchmarks.

Show table
Name Previous Current Ratio Verdict
Block_Unpack-4 4ad8f1 78a38d
90901.00 ns/op 108792.00 ns/op 1.20 🔴

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.

5 participants