dssysolyatin commented on code in PR #4644:
URL: https://github.com/apache/calcite/pull/4644#discussion_r2554303820


##########
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:
   It is a breaking change, but I hope it should not affect anyone. All 
`SqlNode` classes inside `org.apache.calcite.sql.ddl` expose their operands 
through public fields, and none of these nodes implement `setOperand`. Because 
of that, I expect that getOperandList() is not used outside the Calcite 
codebase for these SqlNode types.
   
   For `SqlUnpivot`, I added the SqlLiteral at the end of the operand list.



-- 
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]

Reply via email to