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

RexBuilder.ensureType() does not correctly handle the case when nullability needs to be ignored

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Duplicate
    • Affects Version/s: None
    • Fix Version/s: 1.4.0-incubating
    • Component/s: None
    • Labels:
      None

      Description

      In ensureType() we don't correctly handle the case when nullability should be ignored. We still compare digests of the two types which returns false even when the types are the same but differ only in nullability.

      Created a Pull Request with the description of the issue and a unit test.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                julianhyde Julian Hyde
                Reporter:
                mehant Mehant Baid
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: