michalstutzmann commented on PR #26895: URL: https://github.com/apache/flink/pull/26895#issuecomment-3191509981
> > @snuyanzin @michalstutzmann 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. > > @michalstutzmann I see you have thumbs up this question I am not sure what you mean by this. I see the [docs](https://nightlies.apache.org/flink/flink-docs-release-2.0/docs/dev/table/concepts/overview/#stateful-upgrades-and-evolution) say `In other words: Savepoints are only supported if both the query and the Flink version remain constant.` I assume this statement also effects Calcite version changes. Do you think this change will mean Savepoints are not supported across this change? @davidradl I will check all those things soon and come back to you. Currently on the Janino upgrade and failing tests there (see earlier comment). -- 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]
