Skip to content

parallel-workload: ignore the RTR timeout in ExplainFilterPushdownAction - #38713

Merged
ggevay merged 1 commit into
MaterializeInc:mainfrom
ggevay:gabor/parallel-workload-explain-pushdown-rtr-timeout
Sep 9, 2026
Merged

parallel-workload: ignore the RTR timeout in ExplainFilterPushdownAction#38713
ggevay merged 1 commit into
MaterializeInc:mainfrom
ggevay:gabor/parallel-workload-explain-pushdown-rtr-timeout

Conversation

@ggevay

@ggevay ggevay commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

SelectAction already lists timed out before ingesting the source's visible frontier when real-time-recency query issued as an ignorable error, because a session with real-time recency enabled can legitimately hit the RTR timeout. ExplainFilterPushdownAction runs EXPLAIN FILTER PUSHDOWN FOR SELECT ... on the same sessions and waits for the source the same way, but did not list the error, so an RTR timeout there failed the whole workload: nightly 18001 and 18005 on 2026-08-09 (Parallel Correctness + cancel, Parallel Correctness + 0dt deploy).

One-line addition to the action's errors_to_ignore, mirroring SelectAction. Part of the CI-flake sweep under SS-303, which tracks the RTR timeout itself.

🤖 Generated with Claude Code

SelectAction already treats "timed out before ingesting the source's
visible frontier when real-time-recency query issued" as an ignorable
error, because a session with real-time recency enabled can legitimately
hit the RTR timeout. ExplainFilterPushdownAction issues EXPLAIN FILTER
PUSHDOWN FOR SELECT ... on the same sessions and waits for the source the
same way, but did not list the error, so the RTR timeout failed the whole
workload (nightly 18001 and 18005, 2026-08-09; tracked under SS-303).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@ggevay
ggevay marked this pull request as ready for review September 8, 2026 11:25
@ggevay
ggevay requested a review from a team as a code owner September 8, 2026 11:25
@ggevay ggevay added the T-testing Theme: tests or test infrastructure label Sep 8, 2026
@ggevay
ggevay merged commit fcf8693 into MaterializeInc:main Sep 9, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T-testing Theme: tests or test infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants