Skip to content

feat(etl): rebuild cancellation on refreshed conditional-status stack - #254

Draft
seonghobae wants to merge 20 commits into
repair/durable-job-conditional-status-2e388adfrom
repair/durable-job-cancellation-894e1fd
Draft

feat(etl): rebuild cancellation on refreshed conditional-status stack#254
seonghobae wants to merge 20 commits into
repair/durable-job-conditional-status-2e388adfrom
repair/durable-job-cancellation-894e1fd

Conversation

@seonghobae

Copy link
Copy Markdown
Collaborator

Stack repair

This Draft repairs the first currently invalid durable-job boundary. Fresh ancestry proved current predecessor PR #146 advanced to 894e1fdcb3ad212f35ae0b8c7665b6e495e2eb87, while old cancellation PR #147 remains rooted at historical predecessor 9e4d69e0bb33ab57627c697a5d028b5309eca2bb and is now diverged (ahead 31 / behind 6, merge base 9e4d69e...). The six predecessor commits are confined to the CDC engine health-indicator/test path, while #147's cancellation delta is ETL/docs-only, so this replacement starts from the exact current #146 head rather than rewriting history.

TDD phase

Exact current replacement head f50be481b2b19e6fb566e5430f2d6aac6437fdd4 contains only the first cancellation result contract test. Production EtlJobCancellation and the CANCELLED status contract are intentionally absent on this ancestry, so the expected first hosted result is RED at the missing cancellation production boundary. Setup/import/dependency/runner failure will not count as valid RED.

After valid RED, rebuild the proven cancellation slice through narrow auditable GREEN commits, then restore concurrency/migration/controller/service/documentation tests and prove unique-work equivalence to #147 before considering #147 superseded. No checks/reviews/approvals from #147 transfer. Do not repair downstream replay #148 until this boundary is stack-valid and verified.

No force-push, destructive rebase, ours/theirs shortcut, gate weakening, protection bypass, or cross-repository write is permitted.

@coderabbitai

coderabbitai Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Draft detected.

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: 641c90de-4278-4c7e-ac93-77f54ea16954

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

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@opencode-agent opencode-agent Bot added area: ci-cd CI, GitHub Actions, checks, release, or supply chain area: data Database, schema, migration, ETL, or lineage priority: medium Normal-priority or P2 work status: draft Draft pull request type: feature New or expanded product capability labels Aug 22, 2026
@seonghobae seonghobae added the enhancement New feature or request label Sep 7, 2026 — with ChatGPT Codex Connector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci-cd CI, GitHub Actions, checks, release, or supply chain area: data Database, schema, migration, ETL, or lineage enhancement New feature or request priority: medium Normal-priority or P2 work status: draft Draft pull request type: feature New or expanded product capability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant