I think the concern of breaking plan comparison has appeared more than once. Not only to this proposal but also to others (e.g. replace "$" with "_" in names).
>From a think of another perspective, the widely used practice of string comparison based plan evaluation also reduces the flexibility of making changes in Calcite. We need to carefully think about if a change has a large impact that affects toString(), or something similar. So if Calcite community could offer another solution for downstream projects, and announce that don't guarantee the backward compatibility of toString(), then it could be in a better situation. A few candidate solutions: 1. offer utils that convert a plan represented in string to Relnodes. 2. use RelBuilder to build plans and does comparison. and then just maintain the backward compatibility of the solution. just my two cents. -Rui On Mon, Dec 30, 2019 at 12:20 PM Vladimir Sitnikov < sitnikov.vladi...@gmail.com> wrote: > The change improves slow tests from 80 min to 60, and the changes are > minimal > > Vladimir >
