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

Calls to AbstractRelNode may result in NPE

    Details

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

      Description

      Calls to methods isDistinct and isKey may result in NPE if the value return by the metadata provider is null.

        Issue Links

          Activity

          Hide
          jcamachorodriguez Jesus Camacho Rodriguez added a comment -

          Resolved in release 1.5.0 (2015-11-10)

          Show
          jcamachorodriguez Jesus Camacho Rodriguez added a comment - Resolved in release 1.5.0 (2015-11-10)
          Show
          jcamachorodriguez Jesus Camacho Rodriguez added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/ee0e824 . Sorry about that.
          Hide
          julianhyde Julian Hyde added a comment -

          Commit hash?

          Show
          julianhyde Julian Hyde added a comment - Commit hash?
          Hide
          jcamachorodriguez Jesus Camacho Rodriguez added a comment -

          Pushed to master, thanks for checking the fix Julian Hyde

          Show
          jcamachorodriguez Jesus Camacho Rodriguez added a comment - Pushed to master, thanks for checking the fix Julian Hyde
          Hide
          julianhyde Julian Hyde added a comment -

          My bad - I introduced that problem.

          +1

          Show
          julianhyde Julian Hyde added a comment - My bad - I introduced that problem. +1
          Hide
          jcamachorodriguez Jesus Camacho Rodriguez added a comment -

          Julian Hyde, I created a pull request: https://github.com/apache/incubator-calcite/pull/158

          It's a very simple fix, but since this was introduced in CALCITE-792, could you take a look? Thanks

          Show
          jcamachorodriguez Jesus Camacho Rodriguez added a comment - Julian Hyde , I created a pull request: https://github.com/apache/incubator-calcite/pull/158 It's a very simple fix, but since this was introduced in CALCITE-792 , could you take a look? Thanks

            People

            • Assignee:
              jcamachorodriguez Jesus Camacho Rodriguez
              Reporter:
              jcamachorodriguez Jesus Camacho Rodriguez
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development