Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-273

NotEval incurs NPE with boolean column

    Details

      Description

      tajo> select * from bool_test where col1 is false;
      java.lang.NullPointerException
              at org.apache.tajo.engine.eval.NotEval.postOrder(NotEval.java:91)
              at org.apache.tajo.engine.eval.EvalTreeUtil.findDistinctRefColumns(EvalTreeUtil.java:74)
              at org.apache.tajo.engine.planner.PlannerUtil.canBeEvaluated(PlannerUtil.java:307)
              at org.apache.tajo.engine.planner.rewrite.FilterPushDownRule.visitScan(FilterPushDownRule.java:238)
              at org.apache.tajo.engine.planner.rewrite.FilterPushDownRule.visitScan(FilterPushDownRule.java:38)
              at org.apache.tajo.engine.planner.BasicLogicalPlanVisitor.visitChild(BasicLogicalPlanVisitor.java:93)
      
      1. TAJO-273_3.patch
        7 kB
        Hyunsik Choi
      2. TAJO-273_2.patch
        7 kB
        Hyunsik Choi
      3. TAJO-273.patch
        6 kB
        Hyunsik Choi

        Issue Links

          Activity

          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Tajo-trunk-postcommit #529 (See https://builds.apache.org/job/Tajo-trunk-postcommit/529/)
          TAJO-273: NotEval incurs NPE with boolean column. (hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=b55918bd1dc32a1b034bb67a47114a92bce24b48)

          • tajo-dist/src/main/bin/tajo-daemons.sh
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/eval/NotEval.java
          • LICENSE.txt
          • tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/eval/TestPredicates.java
          • NOTICE.txt
          • CHANGES.txt
          • tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/eval/BasicEvalNodeVisitor.java
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Tajo-trunk-postcommit #529 (See https://builds.apache.org/job/Tajo-trunk-postcommit/529/ ) TAJO-273 : NotEval incurs NPE with boolean column. (hyunsik) (hyunsik: https://git-wip-us.apache.org/repos/asf?p=incubator-tajo.git&a=commit&h=b55918bd1dc32a1b034bb67a47114a92bce24b48 ) tajo-dist/src/main/bin/tajo-daemons.sh tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/eval/NotEval.java LICENSE.txt tajo-core/tajo-core-backend/src/test/java/org/apache/tajo/engine/eval/TestPredicates.java NOTICE.txt CHANGES.txt tajo-core/tajo-core-backend/src/main/java/org/apache/tajo/engine/eval/BasicEvalNodeVisitor.java
          Hide
          hyunsik Hyunsik Choi added a comment -

          Committed to master and branch-0.2.

          Show
          hyunsik Hyunsik Choi added a comment - Committed to master and branch-0.2.
          Hide
          jhkim Jinho Kim added a comment -

          +1
          Looks great for me. I've verified real cluster

          Show
          jhkim Jinho Kim added a comment - +1 Looks great for me. I've verified real cluster
          Hide
          hyunsik Hyunsik Choi added a comment -

          I've uploaded the third patch that fixes some potential bugs.

          Show
          hyunsik Hyunsik Choi added a comment - I've uploaded the third patch that fixes some potential bugs.
          Hide
          hyunsik Hyunsik Choi added a comment -

          This patch fixes the bug.

          Show
          hyunsik Hyunsik Choi added a comment - This patch fixes the bug.

            People

            • Assignee:
              hyunsik Hyunsik Choi
              Reporter:
              hyunsik Hyunsik Choi
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development