You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Add Poll::Pending spill stream test coverage for SMJ async spill paths #22879
The materializing_stream and bitwise_stream async spill read paths (introduced in the refactor) needs test coverage that forces read_spill_as_stream to yield Poll::Pending mid-read to exercise the re-entry logic.
This should become significantly easier once #21882 lands, as a mock spill implementation can inject Poll::Pending without requiring changes to internal stream construction.
Activity
kumarUjjawal commentedon Jun 12, 2026
@pantShrey would you be interested in opening a PR for this? I can review.
pantShrey commentedon Jun 12, 2026
@kumarUjjawal Thank you for offering I would be happy to work on it! Once #21882 merges I will try to land this PR fast as well
pantShrey commentedon Jul 6, 2026
@kumarUjjawal extremely sorry for the delay on this! The PR is finally up here: #23353.
test: add Poll::Pending spill stream coverage for async spill re-entr…