davidzollo commented on PR #10551: URL: https://github.com/apache/seatunnel/pull/10551#issuecomment-5140513823
Addressed the latest review gate by updating the PR description with the broader distributed-state recovery scope. Compatibility evidence for the flagged public API concern: - PhysicalPlan and PhysicalVertex constructor changes only add generic type parameters to the existing Hazelcast IMap arguments. - Java erasure keeps the JVM descriptor as IMap, IMap, so this does not create a different runtime constructor signature for existing callers. - The removed methods in PhysicalPlan / PhysicalVertex are private helpers only (, ), so no public/protected method was removed. Documentation check: - No user-facing config option, default value, connector option, REST/API contract, or serialized checkpoint/split format is changed by this PR. - I therefore updated the PR description rather than adding docs/en or docs/zh option documentation. Local verification boundary remains unchanged: I did not run local compile/test/package/E2E/Docker/service for Apache SeaTunnel; GitHub CI on the PR head is the source of truth. -- 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]
