feat(cost): cost complete streaming summary DAGs - #335
Merged
Merged
Conversation
zzylol
force-pushed
the
feat/cost-streaming-lifecycle-ranking
branch
from
September 3, 2026 16:29
631ac15 to
f88029c
Compare
zzylol
force-pushed
the
feat/cost-streaming-summary-dag
branch
from
September 3, 2026 16:29
0f77c4e to
071c2a6
Compare
zzylol
force-pushed
the
feat/cost-streaming-lifecycle-ranking
branch
from
September 3, 2026 16:45
f88029c to
77658a1
Compare
zzylol
force-pushed
the
feat/cost-streaming-summary-dag
branch
2 times, most recently
from
September 3, 2026 16:50
8e30f91 to
555113c
Compare
zzylol
force-pushed
the
feat/cost-streaming-lifecycle-ranking
branch
from
September 4, 2026 00:12
089c8c1 to
40db2f1
Compare
zzylol
force-pushed
the
feat/cost-streaming-summary-dag
branch
from
September 4, 2026 00:35
1f80380 to
0e9f092
Compare
zzylol
force-pushed
the
feat/cost-streaming-lifecycle-ranking
branch
from
September 4, 2026 02:25
40db2f1 to
0219280
Compare
zzylol
force-pushed
the
feat/cost-streaming-summary-dag
branch
from
September 4, 2026 02:26
0e9f092 to
d02bf12
Compare
zzylol
force-pushed
the
feat/cost-streaming-lifecycle-ranking
branch
from
September 4, 2026 02:50
0219280 to
d72457a
Compare
zzylol
force-pushed
the
feat/cost-streaming-summary-dag
branch
from
September 4, 2026 03:00
d02bf12 to
5f71dee
Compare
zzylol
force-pushed
the
feat/cost-streaming-lifecycle-ranking
branch
from
September 4, 2026 03:26
48c15ca to
6e60fab
Compare
zzylol
force-pushed
the
feat/cost-streaming-summary-dag
branch
from
September 4, 2026 03:29
5f71dee to
4cd0ee2
Compare
zzylol
force-pushed
the
feat/cost-streaming-lifecycle-ranking
branch
from
September 4, 2026 04:01
6e60fab to
b4c5480
Compare
zzylol
force-pushed
the
feat/cost-streaming-summary-dag
branch
from
September 4, 2026 04:01
4cd0ee2 to
673e74a
Compare
zzylol
changed the base branch from
feat/cost-streaming-lifecycle-ranking
to
main
September 4, 2026 11:27
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Summing per-summary estimates can miss merge, subtract, delete, join, readout, shared-state, and retained-input work in a nested SummaryExpr DAG.
What
Estimate the complete selected summary DAG and reject incomplete or contradictory physical evidence.
How
Before this PR
Lifecycle costing could rely on partial per-state totals and omit parent or shared operations.
After this PR
A candidate is available only when its complete streaming SummaryExpr DAG has consistent evidence.
Stack
Depends on #334. The next PR integrates complete estimates into global planner selection.
Module boundary
Summary-maintenance evidence is separated from its planner adapter. Raw streaming baselines now use the shared
EvidenceBackedPhysicalDag; the duplicate streaming-only physical DAG/statistics container has been removed.