Skip to content

Add stop conditions for AutoReviewPoller and DownstreamPoller queries#223

Merged
mawelborn merged 1 commit into7.2.3-releasefrom
mawelborn/polling-stop-conditions
Jan 21, 2026
Merged

Add stop conditions for AutoReviewPoller and DownstreamPoller queries#223
mawelborn merged 1 commit into7.2.3-releasefrom
mawelborn/polling-stop-conditions

Conversation

@mawelborn
Copy link
Contributor

@mawelborn mawelborn commented Jan 12, 2026

Part of a WIP v7.2.3 release.

This PR updates the submission list filters for AutoReviewPoller and DownstreamPoller to skip submissions that have been deleted. This prevents them from repeatedly retrying submissions that can never be processed, rather than requiring manual intervention as before.

It also updates the AutoReviewPoller filter to skip retrieved submissions. This creates a way to stop auto review from retrying a submission that can't be processed for other reasons--eg.g. an invalid result file or other runtime error. Where before the submission had to be failed in the DB, now it can be marked retrieved via API or UI to stop AR from trying to process it.


Note

Prevents pollers from retrying unprocessable submissions by tightening GraphQL filters.

  • Updates SubmissionIdsPendingAutoReview to require status: PENDING_AUTO_REVIEW and also exclude { filesDeleted: false, retrieved: false }
  • Updates SubmissionIdsPendingDownstream to exclude { filesDeleted: false, retrieved: false } and retain OR on { status: COMPLETE | FAILED }
  • Refactors filters.AND into list form in both queries

Written by Cursor Bugbot for commit ba123a3. This will update automatically on new commits. Configure here.

@mawelborn mawelborn merged commit 60718d3 into 7.2.3-release Jan 21, 2026
3 checks passed
@mawelborn mawelborn deleted the mawelborn/polling-stop-conditions branch January 21, 2026 19:41
@mawelborn mawelborn mentioned this pull request Feb 3, 2026
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.

2 participants