You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@calcite.apache.org by "YUAN (Jira)" <ji...@apache.org> on 2022/11/11 06:41:00 UTC

[jira] [Created] (CALCITE-5377) RelFieldTrimmer support Sort with dynamic param

YUAN created CALCITE-5377:
-----------------------------

             Summary: RelFieldTrimmer support Sort with dynamic param
                 Key: CALCITE-5377
                 URL: https://issues.apache.org/jira/browse/CALCITE-5377
             Project: Calcite
          Issue Type: Improvement
          Components: core
    Affects Versions: 1.32.0
            Reporter: YUAN


If Sort rel node used dynamic param, RelFieldTrimmer can not trim fields as expected.

The behavior makes ProjectableFilterableTable can not get correct projects.

Related code:
{code:java}
// org.apache.calcite.sql2rel.RelFieldTrimmer#trimFields(org.apache.calcite.rel.core.Sort, org.apache.calcite.util.ImmutableBitSet, java.util.Set<org.apache.calcite.rel.type.RelDataTypeField>)
// leave the Sort unchanged in case we have dynamic limits
if (sort.offset instanceof RexDynamicParam
    || sort.fetch instanceof RexDynamicParam) {
  return result(sort, inputMapping);
} {code}
I think we can support dynamic param without breaking anything.



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