dssysolyatin commented on code in PR #4644:
URL: https://github.com/apache/calcite/pull/4644#discussion_r2649908783
##########
site/_docs/history.md:
##########
@@ -49,6 +49,12 @@ other software versions as specified in gradle.properties.
#### Breaking Changes
{: #breaking-1-42-0}
+* [<a
href="https://issues.apache.org/jira/browse/CALCITE-7301";>CALCITE-7301</a>]
Review Comment:
@mihaibudiu I personally would prefer to include this as a breaking change,
including the changes you made in your `ddl` related PR, which you submitted
separately.
The number of arguments in the operator list has changed, and this is a
runtime breaking change. Existing code may assume different behavior, for
example:
```
if (!operandList.isEmpty) {
// show user first operand or another crazy cases
}
```
--
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]
