snuyanzin commented on PR #26895: URL: https://github.com/apache/flink/pull/26895#issuecomment-3192771305
>I see changes in the tests around the SQL and am worried whether this could mean there is a backwards compatibility issue in some way. For example would restoring savepoints across the Calcite version change, work? Is there a test for this? I assume a more efficient optimization at a higher Calcite level does not cause a compatibility issue - though the explain output may change. It would be great if you could reassure me these worries are unfounded. if a change (not only) Calcite upgrade leads to change (not only description change) in CompiledPlan store/restore tests then it is most probably a breaking change. If a change leads to change in optimized plan tests then it is at least questionable... Of course our tests do not cover all the cases and could miss something -- 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]
