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

      Hide
      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
      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