Uploaded image for project: 'Hive'
  1. Hive
  2. HIVE-26652

HiveSortPullUpConstantsRule produces an invalid plan when pulling up constants for nullable fields

    XMLWordPrintableJSON

Details

    Description

      The rule pulls up constants without checking/adjusting nullability to match that of the field type.

      Here is the stack-trace when a nullable type is involved:

      java.lang.AssertionError: type mismatch:
      ref:
      JavaType(class java.lang.Integer)
      input:
      JavaType(int) NOT NULL    at org.apache.calcite.util.Litmus$1.fail(Litmus.java:31)
          at org.apache.calcite.plan.RelOptUtil.eq(RelOptUtil.java:2167)
          at org.apache.calcite.rex.RexChecker.visitInputRef(RexChecker.java:125)
          at org.apache.calcite.rex.RexChecker.visitInputRef(RexChecker.java:57)
          at org.apache.calcite.rex.RexInputRef.accept(RexInputRef.java:112)
          at org.apache.calcite.rel.core.Project.isValid(Project.java:215)
          at org.apache.calcite.rel.core.Project.<init>(Project.java:94)
          at org.apache.calcite.rel.core.Project.<init>(Project.java:100)
          at org.apache.hadoop.hive.ql.optimizer.calcite.reloperators.HiveProject.<init>(HiveProject.java:58)
          at org.apache.hadoop.hive.ql.optimizer.calcite.reloperators.HiveProject.copy(HiveProject.java:106)
          at org.apache.calcite.rel.core.Project.copy(Project.java:126)
          at org.apache.hadoop.hive.ql.optimizer.calcite.rules.HiveSortPullUpConstantsRule$HiveSortPullUpConstantsRuleBase.onMatch(HiveSortPullUpConstantsRule.java:195)
          at org.apache.calcite.plan.AbstractRelOptPlanner.fireRule(AbstractRelOptPlanner.java:333)
          at org.apache.calcite.plan.hep.HepPlanner.applyRule(HepPlanner.java:542)
          at org.apache.calcite.plan.hep.HepPlanner.applyRules(HepPlanner.java:407)
          at org.apache.calcite.plan.hep.HepPlanner.executeInstruction(HepPlanner.java:243)
          at org.apache.calcite.plan.hep.HepInstruction$RuleInstance.execute(HepInstruction.java:127)
          at org.apache.calcite.plan.hep.HepPlanner.executeProgram(HepPlanner.java:202)
          at org.apache.calcite.plan.hep.HepPlanner.findBestExp(HepPlanner.java:189)
          at org.apache.hadoop.hive.ql.optimizer.calcite.rules.TestHiveSortExchangePullUpConstantsRule.test(TestHiveSortExchangePullUpConstantsRule.java:104)
          at org.apache.hadoop.hive.ql.optimizer.calcite.rules.TestHiveSortExchangePullUpConstantsRule.testNullableFields(TestHiveSortExchangePullUpConstantsRule.java:156)
          at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
          at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
          at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
          at java.lang.reflect.Method.invoke(Method.java:498)
          at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
          at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
          at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:56)
          at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
          at org.mockito.internal.runners.DefaultInternalRunner$1$1.evaluate(DefaultInternalRunner.java:54)
          at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
          at org.junit.runners.BlockJUnit4ClassRunner$1.evaluate(BlockJUnit4ClassRunner.java:100)
          at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:366)
          at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:103)
          at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:63)
          at org.junit.runners.ParentRunner$4.run(ParentRunner.java:331)
          at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:79)
          at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:329)
          at org.junit.runners.ParentRunner.access$100(ParentRunner.java:66)
          at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:293)
          at org.junit.runners.ParentRunner$3.evaluate(ParentRunner.java:306)
          at org.junit.runners.ParentRunner.run(ParentRunner.java:413)
          at org.mockito.internal.runners.DefaultInternalRunner$1.run(DefaultInternalRunner.java:99)
          at org.mockito.internal.runners.DefaultInternalRunner.run(DefaultInternalRunner.java:105)
          at org.mockito.internal.runners.StrictRunner.run(StrictRunner.java:40)
          at org.mockito.junit.MockitoJUnitRunner.run(MockitoJUnitRunner.java:163)
          at org.junit.runner.JUnitCore.run(JUnitCore.java:137)
          at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69)
          at com.intellij.rt.junit.IdeaTestRunner$Repeater$1.execute(IdeaTestRunner.java:38)
          at com.intellij.rt.execution.junit.TestsRepeater.repeat(TestsRepeater.java:11)
          at com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:35)
          at com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:235)
          at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) 

      The solution is to check nullability and add a cast when the field is nullable, since the constant's type is not.

      Attachments

        Issue Links

          Activity

            People

              asolimando Alessandro Solimando
              asolimando Alessandro Solimando
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 0.5h
                  0.5h