DanielLeens commented on PR #12284: URL: https://github.com/apache/seatunnel/pull/12284#issuecomment-5651443191
Status update after the branch was merged with `dev` (`06daecd8822`): - #11503 is now merged and part of this branch, so the restore-after-failover dependency is resolved. Commit `e7bccdfb0b2` handles `RestoreTableSchemaEvent` explicitly in the SQL transform (re-evaluate on the restored table, derive no DDL, set `changeAfter` to the produced table, return the same instance) and fails fast with `TRANSFORM_COMMON-09` for any other table-level event without column sub-events. The merge kept this implementation over the four-line #11503 hunk in `SQLTransform`, which had the same effect; `SQLTransformTest.testRestoreSchemaEventRebuildsProducedSchema` from #11503 runs against it. - The E2E now covers savepoint and restore through the SQL transform: the `select *` job runs the same drop-while-stopped restore cycle as the no-transform test; the projection job adds a rows-only restore cycle, an operand type change while stopped (restore event, then the translated modify) and a DDL applied to the restored job. The PR description is updated accordingly, including the fixture changes from the first review round (FLOAT to DECIMAL(12,3), sink starting with the derived DOUBLE). - CI: the Build runs for the two earlier heads were cancelled by the following pushes; the run for this head is pending in the fork. -- 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]
