fix(cdc): expose truthful target execution capabilities - #249
Closed
seonghobae wants to merge 4 commits into
Closed
Conversation
Contributor
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
This was referenced Aug 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes the Java-SPI portion of #202 without racing the active
CdcControllerwriter. The branch was cut from protecteddevelop@c2511f6b9d716ef9ab8de60a91fffd826714a4c5; the independently resolved live protected tip is nowdevelop@942d164ceb7af497bb28a95f842db96429d1bdc4. Exact source head remainsf2fd61a50c028535ec30a92d1751e6bf419b1fc4. The intervening protected change (#245) is confined to config-alias sources/resources/tests and does not overlap this PR's four CDC SPI/test paths, but this branch has not been refreshed onto that live base and no evidence transfers across base movement.Implemented scope
CdcTargetConnectorexposes immutable machine-readable execution capabilities instead of overloadingscaffoldOnly().processed_dataJDBC-replica path with canonical SPI writes still unwired./api/cdc/targetsand/statusprojection remains a separate follow-up after the controller writer lane clears; this PR must not claim that portion complete.Exact observed validation
f2fd61a50c028535ec30a92d1751e6bf419b1fc4;31430806262checked out synthetic mergebd74de4e72a5c1915f2a5b2f94a6093c29ecd7a6(f2fd61a...merged into historical basec2511f6...), not the literal source head and not the current live protected base;CdcTargetCapabilityTest, and gateway 3/3 tests; config-server and eureka-server had no tests;Analyzed bundle 'etl-service' with 0 classesfollowed byAll coverage checks have been met, so the observed coverage gate is vacuous rather than qualifying 100% owned-production evidence;Remaining boundary
Keep Draft. Before any merge, rebuild or non-destructively refresh from the exact current protected ancestry and require literal-source exact-head/current-base validation, non-vacuous owned-production statement/branch coverage, complete applicable vulnerability/dependency scanning, all required security/SBOM/provenance gates, zero valid unresolved findings, and qualifying independent non-author formal approval. Predecessor-head, stale-base, synthetic-merge, status-only, incomplete-scanner, skipped-required, neutral, or author-only evidence is not merge authority.