Skip to content

feat: remove token _all_ - #591

Open
eqimd wants to merge 11 commits into
mainfrom
586-remove-token-all
Open

eqimd wants to merge 11 commits into
mainfrom
586-remove-token-all

Conversation

@eqimd

@eqimd eqimd commented Oct 22, 2026

Copy link
Copy Markdown
Contributor

Description

Remove token _all_ from sealed fraction and proxy->store interaction

Fixes #(586)


@codecov-commenter

codecov-commenter commented Oct 22, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 95.83333% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.10%. Comparing base (2d0d610) to head (697ae5c).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
frac/processor/batch_eval_tree.go 88.23% 2 Missing ⚠️
node/node_range.go 93.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #591      +/-   ##
==========================================
+ Coverage   79.78%   80.10%   +0.31%     
==========================================
  Files         259      261       +2     
  Lines       19510    19747     +237     
==========================================
+ Hits        15566    15818     +252     
+ Misses       3944     3929      -15     

☔ 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.

@eqimd eqimd self-assigned this Oct 22, 2026
@eqimd
eqimd requested review from cheb0 and eguguchkin October 26, 2026 07:06
@eguguchkin eguguchkin modified the milestones: v0.82.0, v0.83.0 Oct 26, 2026
Comment thread frac/processor/eval_tree.go Resolved
Comment thread frac/processor/eval_tree.go Resolved
Comment thread cmd/fraction/decoding_suites_test.go Resolved
Comment thread frac/fraction_test.go Resolved
Comment thread frac/fraction_test.go Resolved
Comment thread frac/fraction_test.go Resolved
Comment thread frac/active_indexer.go Resolved
Comment thread frac/fraction_test.go Resolved
@eqimd
eqimd force-pushed the 586-remove-token-all branch 2 times, most recently from 4745fd5 to da92f0a Compare October 30, 2026 11:31
@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
Indexer-4 64e2ef 3cf392
564879420.00 B/op 764050221.00 B/op 1.35 🔴

Comment thread tests/docker-compose.yml Resolved
Comment thread node/batch_ops_test.go Resolved
@github-actions

github-actions Bot commented Nov 2, 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
OrTree/size=1000000-4 114c79 216f49
5.97 ns/op 6.66 ns/op 1.12 🔴

This branch has not been deployed

No deployments
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.

4 participants