>With the size and complexity of Calcite project,

... nobody can really understand anything, so there's no point of waiting
for +1 :)

Julian, Xiening you both are right.

I would add a small note that "easy to understand" titles significantly
help for reviewers.
Here's a recent case: https://github.com/apache/calcite/pull/1814
[CALCITE-3804] Use RelCollation interface in RelCollationImpl compareTo and
satisfies methods

I even tried to reverse-engineer the intention behind the change, however,
I just stopped.
If only the author provided that intention in the PR commit...

Vladimir

Reply via email to