[ 
https://issues.apache.org/jira/browse/CALCITE-6968?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18070275#comment-18070275
 ] 

Dmitry Sysolyatin commented on CALCITE-6968:
--------------------------------------------

sourceSelect is used only internally. It does not use during parse/unparse and 
set only inside `SqlValidatorImpl#createSourceSelectForUpdate(SqlUpdate)` and 
is used for `SqlToRelConverter`. I don't think it should be exposed inside 
getOperandList

> SqlUpdate#getOperandList omits sourceSelect operand
> ----------------------------------------------------
>
>                 Key: CALCITE-6968
>                 URL: https://issues.apache.org/jira/browse/CALCITE-6968
>             Project: Calcite
>          Issue Type: Bug
>          Components: core
>    Affects Versions: 1.39.0
>            Reporter: Wei.Zhou
>            Assignee: hongyu guo
>            Priority: Major
>              Labels: pull-request-available
>
> @SuppressWarnings("nullness")
> @Override public List<@Nullable SqlNode> getOperandList() {
> return ImmutableNullableList.of(targetTable, targetColumnList,
> sourceExpressionList, condition, alias);
> }
> Maybe this method return operands missing 'sourceSelect'



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to