Skip to content

Iterative Execution 4a#6755

Draft
gatesn wants to merge 2 commits intongates/execution-3from
ngates/execution-4a
Draft

Iterative Execution 4a#6755
gatesn wants to merge 2 commits intongates/execution-3from
ngates/execution-4a

Conversation

@gatesn
Copy link
Contributor

@gatesn gatesn commented Mar 2, 2026

No description provided.

Copy link
Contributor Author

gatesn commented Mar 2, 2026

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 2, 2026

Merging this PR will degrade performance by 10.2%

❌ 1 regressed benchmark
✅ 953 untouched benchmarks
⏩ 1466 skipped benchmarks1

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation map_each[BufferMut<i32>, 128] 770.6 ns 858.1 ns -10.2%

Comparing ngates/execution-4a (e7e9518) with ngates/execution-3 (b5b1652)

Open in CodSpeed

Footnotes

  1. 1466 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@gatesn gatesn changed the title feat: migrate Zstd decompressors to iterative execute Iterative Execution 4a Mar 2, 2026
@gatesn gatesn changed the base branch from ngates/execution-3 to graphite-base/6755 March 2, 2026 19:04
@gatesn gatesn force-pushed the ngates/execution-4a branch from 08d7d8e to 1711dab Compare March 2, 2026 19:05
@gatesn gatesn changed the base branch from graphite-base/6755 to ngates/execution-3 March 2, 2026 19:05
@gatesn gatesn added the changelog/feature A new feature label Mar 2, 2026
@gatesn gatesn changed the base branch from ngates/execution-3 to graphite-base/6755 March 2, 2026 19:19
@gatesn gatesn force-pushed the graphite-base/6755 branch from 4223d1f to 2b9697c Compare March 2, 2026 19:19
@gatesn gatesn force-pushed the ngates/execution-4a branch from 1711dab to d5bf556 Compare March 2, 2026 19:19
@gatesn gatesn changed the base branch from graphite-base/6755 to ngates/execution-3 March 2, 2026 19:19
@gatesn gatesn force-pushed the ngates/execution-4a branch 2 times, most recently from f9a335a to 70cd549 Compare March 2, 2026 20:13
@gatesn gatesn force-pushed the ngates/execution-3 branch from 5cd1e99 to 6c8dcdc Compare March 2, 2026 20:13
gatesn and others added 2 commits March 2, 2026 15:43
ZstdVTable and ZstdBuffersVTable no longer recursively call
.execute() after decompression. Instead they return Done with the
decompressed intermediate, letting the scheduler re-enter execution
on the result naturally.

Signed-off-by: Nicholas Gates <nick@nickgates.com>

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Nicholas Gates <nick@nickgates.com>

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@gatesn gatesn force-pushed the ngates/execution-4a branch from 70cd549 to e7e9518 Compare March 2, 2026 20:43
@gatesn gatesn force-pushed the ngates/execution-3 branch from 6c8dcdc to b5b1652 Compare March 2, 2026 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/feature A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant