Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-127

EnumerableCalcRel can't support 3+ AND conditions on 0.4.17

        Details

        • Type: Bug
        • Status: Closed
        • Priority: Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: None
        • Component/s: None
        • Labels:

          Description

          When I migrate to 0.4.17, I find that EnumerableCalcRel can't support 3+ AND condition. The root cause is as below:

          1. RexToLixTranslator.translateCondition() call RexToLixTranslator.translate() with RexImpTable.NullAs.FALSE

          2. RexImpTable.createImplementor() (line 252) will call Expressions.foldAnd() only when NullAs.TRUE & NullAs.NOT_POSSIBLE. Otherwise, it only process 2 condition.

          It break many our test cases.

          ---------------- Imported from GitHub ----------------
          Url: https://github.com/julianhyde/optiq/issues/127
          Created by: jiangxuchina
          Labels: bug,
          Assignee: hbutani
          Created at: Mon Feb 10 16:09:08 CET 2014
          State: closed

            Activity

            Hide
            Permalink
            github-import GitHub Import added a comment -

            [Date: Mon Feb 10 19:57:40 CET 2014, Author: julianhyde]

            I have reproduced this. The 3rd (and subsequent) conditions in the AND are ignored.

            Show
            github-import GitHub Import added a comment - [Date: Mon Feb 10 19:57:40 CET 2014, Author: julianhyde ] I have reproduced this. The 3rd (and subsequent) conditions in the AND are ignored.

              People

              • Assignee:
                Unassigned
                Reporter:
                github-import GitHub Import
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development