Daishuyuan commented on PR #4320:
URL: https://github.com/apache/flink-cdc/pull/4320#issuecomment-4714116031

   @leonardBang, I pushed an update to fix the SQL Server test compilation 
under the Flink 2 profile.
   
   This update replaces the removed Flink 1.x APIs used by SQL Server pipeline 
tests:
   - `RestartStrategies` -> `RestartStrategyUtils`
   - `SavepointConfigOptions.SAVEPOINT_PATH` -> 
`StateRecoveryOptions.SAVEPOINT_PATH`
   - `triggerSavepoint(path)` -> `triggerSavepoint(path, 
SavepointFormatType.DEFAULT)`
   - collect result setup now follows the existing MySQL/Postgres/Oracle Flink 
2 compatibility pattern.
   
   For the other failed checks, they look unrelated to this PR:
   - `Source Unit Tests 2.x` failed in 
`OceanBaseFailoverITCase.testJobManagerFailoverFromLatestOffset` with a result 
assertion mismatch.
   - `Pipeline E2E Tests` failed in 
`MySqlToIcebergE2eITCase.testSyncWholeDatabase` with a sink result assertion 
mismatch.
   - The SQL Server-related Flink 2 compilation failure should be covered by 
this update.
   
   Could you please rerun the CI? Thanks.


-- 
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]

Reply via email to