The change in CALCITE-3713 make a radical change to the plan digest, but it only brings in little promotion(at least from Calcite tests and Flink cases)
During the upgrade of Flink Calcite version to 1.22.0-SNAPSHOT, the 2000+ plan changes to due to this patch, it is hard for me to figure out the reason of the change: because of this patch, or because of other bug-fix/promotion. Even though I change the plan verification to EXPLAIN_ATTRIBUTE, there are still 500+ plan diffs. This makes the Flink Calcite upgrade almost impossible to go on and I’m burdened out of it. So, PLEASE, if possible, can we REVERT this patch, 2000+ plan changes for no promotion(useless) (at lease for Flink). I did show my options when discussing if the patch should be merged, me and Julian all gave -1 for that but the code still merged, sigh ! [1] https://ponymail-vm.apache.org/_GUI_/thread.html/54bf3ed733eb7e725ce3ea397334aad8f1323ead13e450b1753b1521%40%3Cdev.calcite.apache.org%3E Best, Danny Chan
