[
https://issues.apache.org/jira/browse/CALCITE-6968?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18070275#comment-18070275
]
Dmitry Sysolyatin edited comment on CALCITE-6968 at 4/1/26 9:06 AM:
--------------------------------------------------------------------
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
was (Author: dmsysolyatin):
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)
