imbajin opened a new pull request, #822:
URL: https://github.com/apache/hugegraph-toolchain/pull/822

   ## Purpose of the PR
   
   The CDC recovery fixture copied only the launcher, so its shared parameter 
helper was missing and validation stopped before Flink submission. Copy both 
existing scripts into the fixture bin directory. This follows [ASF 
#819](https://github.com/apache/hugegraph-toolchain/pull/819) and changes only 
the test fixture.
   
   ## Verifying these changes
   
   The local control reproduces the missing-helper failure with the original 
fixture layout. With this change, the actual fixture launcher loads the native 
helper and reaches a mock Flink executable with argument boundaries preserved; 
a missing value still prevents submission. Existing launcher checks, Python 
helper tests, Python compilation and `git diff --check` pass.
   
   The native Docker CDC snapshot/recovery scenario now passes in [the source 
PR's 
CI](https://github.com/hugegraph/hugegraph-toolchain/actions/runs/38633061648/job/116090150916),
 together with its license checks.
   
   ## Does this PR potentially affect the following parts?
   
   - [ ] Dependencies
   - [ ] Modify configurations
   - [ ] The public API
   - [ ] Other affects
   - [x] Nope: test-fixture layout only
   
   ## Documentation Status
   
   - [ ] `Doc - TODO`
   - [ ] `Doc - Done`
   - [x] `Doc - No Need`: test fixture only; product behavior and documentation 
are unchanged.
   
   Paired source PR: https://github.com/hugegraph/hugegraph-toolchain/pull/74
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to