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
        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