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

RelBuilder.scan() throws NPE if relation does not exist

    Details

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

      Description

      RelBuilder.scan() throws NPE if relation does not exist. The fix is simple to add a null check and throw an appropriate exception.

      Pull request: https://github.com/apache/incubator-calcite/pull/159

        Activity

        Hide
        julianhyde Julian Hyde added a comment -

        A few things to fix in the PR but looks mostly good.

        Show
        julianhyde Julian Hyde added a comment - A few things to fix in the PR but looks mostly good.
        Hide
        andygrove73 Andy Grove added a comment -

        Thanks. I've just submitted changes to address these issues.

        Show
        andygrove73 Andy Grove added a comment - Thanks. I've just submitted changes to address these issues.
        Show
        julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/57e1b4c1 . Thanks for the patch, Andy Grove !
        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)

          People

          • Assignee:
            andygrove Andy Grove
            Reporter:
            andygrove73 Andy Grove
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 5m
              5m
              Remaining:
              Remaining Estimate - 5m
              5m
              Logged:
              Time Spent - Not Specified
              Not Specified

                Development