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

Change visibility of constructor in metadata providers for size, memory, parallelism

        Details

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

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          julianhyde Julian Hyde added a comment -

          Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/b659d0f3.

          Note that you can chain metadata providers. So, you could create say HiveRelMdMemory, a metadata provider that implements methods for Hive relational expressions, and chain it with RelMdMemory, the metadata provider that implements the core relational expressions.

          However, sub-classing is also reasonable, so I am accepting this pull request.

          Show
          julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/b659d0f3 . Note that you can chain metadata providers. So, you could create say HiveRelMdMemory, a metadata provider that implements methods for Hive relational expressions, and chain it with RelMdMemory, the metadata provider that implements the core relational expressions. However, sub-classing is also reasonable, so I am accepting this pull request.
          Hide
          Permalink
          julianhyde Julian Hyde added a comment -

          Closing now that 1.1.0-incubating has been released.

          Show
          julianhyde Julian Hyde added a comment - Closing now that 1.1.0-incubating has been released.

            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