You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@calcite.apache.org by "Steven Talbot (Jira)" <ji...@apache.org> on 2020/03/25 23:45:00 UTC

[jira] [Created] (CALCITE-3877) Null lowerBound/upperBond in RexWindow causes NPE in RexShuttle

Steven Talbot created CALCITE-3877:
--------------------------------------

             Summary: Null lowerBound/upperBond in RexWindow causes NPE in RexShuttle
                 Key: CALCITE-3877
                 URL: https://issues.apache.org/jira/browse/CALCITE-3877
             Project: Calcite
          Issue Type: Bug
            Reporter: Steven Talbot


Here's a test that can be dropped into RexShuttleTest.java

 
{code:java}
@Test public void handlesNullBoundsInRexWindow() {
  final RelBuilder builder = RelBuilder.create(RelBuilderTest.config().build());
  final RexNode over = builder.getRexBuilder().makeOver(
      builder.getTypeFactory().createSqlType(SqlTypeName.BIGINT),
      SqlStdOperatorTable.ROW_NUMBER,
      ImmutableList.of(),
      ImmutableList.of(),
      ImmutableList.of(),
      null,
      null,
      true,
      true,
      true,
      true,
       true
  );
  assertThat(over.accept(new RexShuttle()), is(over));
}
{code}
 

 

That will raise an NPE. Fix is to handle NULLs when doing the child accept on the RexWindowBound[s] in RexShuttle.visitWindow.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)