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

Allow NULL for ARRAY constructor

        Details

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

          Description

          Type of NULL can be inferred as a component type of Array.

            Issue Links

            links to

            Web Link Pull request

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              michaelmior Michael Mior added a comment - - edited

              Merged in f9f2ec149

              Show
              michaelmior Michael Mior added a comment - - edited Merged in f9f2ec149
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              Force-pushed https://git-wip-us.apache.org/repos/asf?p=calcite.git;a=commit;h=32747d794bc7dae41b98bf508f350b4be878e67e to change the commit message.

              After this commit, the validator is OK with null literals in arrays but sql-to-rel isn't. I tried to run the following query but it gave errors: 

              select array[1,null,2] as a from (values (1))

              Rather than re-open this case, let's get that working as part of CALCITE-1804.

              Show
              julianhyde Julian Hyde added a comment - Force-pushed https://git-wip-us.apache.org/repos/asf?p=calcite.git;a=commit;h=32747d794bc7dae41b98bf508f350b4be878e67e to change the commit message. After this commit, the validator is OK with null literals in arrays but sql-to-rel isn't. I tried to run the following query but it gave errors: select array[1, null ,2] as a from (values (1)) Rather than re-open this case, let's get that working as part of CALCITE-1804 .
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              I improved the fix in http://git-wip-us.apache.org/repos/asf/calcite/commit/4069013f. Now SQL such as the above query runs end-to-end, and similar queries that use MULTISET.

              Show
              julianhyde Julian Hyde added a comment - I improved the fix in http://git-wip-us.apache.org/repos/asf/calcite/commit/4069013f . Now SQL such as the above query runs end-to-end, and similar queries that use MULTISET.
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

              Marking closed as of release 1.13.0 (2017-06-26).

              Show
              julianhyde Julian Hyde added a comment - Marking closed as of release 1.13.0 (2017-06-26).

                People

                • Assignee:
                  michaelmior Michael Mior
                  Reporter:
                  ankit@apache.org Ankit Singhal
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development