[
https://issues.apache.org/jira/browse/CALCITE-5915?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Vladimir Steshin updated CALCITE-5915:
--------------------------------------
Description:
Not sure if it is a bug becuase the query plans in this case might not be final
and require further expanding, but the SQL hints might not be propagated to
subqueries when SqlToRelConverter#withExpand==false. This happens because hints
are pushed down with _RelOptUtil#RelHintPropagateShuttle_ which travese through
_RelNode#getInputs()_. But what if node is not accessible as other node’s input
like _LogicalFilter#condition_? Then the shuttle skips such nodes. I found tah
may happen in qubqueries.
Test attached.
was:
The SQL hints might not be propagated in decorrelated subqueries when
SqlToRelConverter#withExpand==false. This happens because hints are pushed down
with _RelOptUtil#RelHintPropagateShuttle_ which travese through
_RelNode#getInputs()_. But what if node is not accessible as other node’s input
like _LogicalFilter#condition_? Then the shuttle skips such nodes. I found tah
may happen in qubqueries.
Test attached.
> Missing SQL hints in not-expanded subqueries.
> ----------------------------------------------
>
> Key: CALCITE-5915
> URL: https://issues.apache.org/jira/browse/CALCITE-5915
> Project: Calcite
> Issue Type: Bug
> Reporter: Vladimir Steshin
> Priority: Minor
> Attachments: SqlHintsInSubqueriesWithDisabledExpanding.java
>
>
> Not sure if it is a bug becuase the query plans in this case might not be
> final and require further expanding, but the SQL hints might not be
> propagated to subqueries when SqlToRelConverter#withExpand==false. This
> happens because hints are pushed down with
> _RelOptUtil#RelHintPropagateShuttle_ which travese through
> _RelNode#getInputs()_. But what if node is not accessible as other node’s
> input like _LogicalFilter#condition_? Then the shuttle skips such nodes. I
> found tah may happen in qubqueries.
> Test attached.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
