Skip to content

superseded: add conditional durable-job status validation - #128

Closed
seonghobae wants to merge 11 commits into
feat/durable-job-poll-advisoryfrom
feat/durable-job-conditional-status
Closed

superseded: add conditional durable-job status validation#128
seonghobae wants to merge 11 commits into
feat/durable-job-poll-advisoryfrom
feat/durable-job-conditional-status

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Superseded by PR #131

This pull request is closed because it depended on superseded ancestry-diverged PR #126 and GitHub reported it non-mergeable. PR #131 reconstructs the same standards-based conditional status behavior from replacement PR #130 exact head f7502de7402fc63868d110d3d7a87b728078fb35 through connector-backed auditable commits.

This PR and its branch remain preserved as the original fail-first evidence, including the null-versus-empty validator collision regression. The replacement does not force-push this branch, discard predecessor work, or use ours/theirs conflict heuristics.

Exact superseded head: f675b6e976efbd8821bcf889ecb0133c60c6e53d.
Replacement PR: #131.

No check, review, or approval from this superseded exact head transfers to PR #131.

@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: b179c1c5-522d-4002-99dd-82ce3daf2148

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@seonghobae
seonghobae marked this pull request as ready for review August 5, 2026 11:22

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@seonghobae seonghobae changed the title feat(etl): add conditional durable-job status validation superseded: add conditional durable-job status validation Aug 5, 2026
@seonghobae seonghobae closed this Aug 5, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by repaired stacked conditional-validation PR #131, which preserves the corrected predecessor lineage and original fail-first null-versus-empty history while carrying the complete owner-safe ETag implementation. Keep this predecessor closed so one validation branch proceeds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant