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

Add RelDataTypeSystem plugin, allowing different max precision of a DECIMAL

    Details

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

      Description

      Hive has different max scale/precision than Optiq.

      This patch just changes it to Hive values.
      This is most probably not how to address this. May be we need some setter methods or factory for this.

      This patch is just to capture the type issues surrounding precision/scale in Optiq w.r.t Hive.

      1. OPTIQ-413.patch
        1 kB
        Laljo John Pullokkaran

        Activity

        Hide
        julianhyde Julian Hyde added a comment -

        Adding interface RelDataTypeSystem. The constructor of RelDataTypeFactoryImpl now takes a RelDataTypeSystem. Most applications will use RelDataTypeSystem.DEFAULT if you like.

        Also adding a new connection property, "typeSystem", and adding a method FrameworkConfig.getTypeSystem().

        Show
        julianhyde Julian Hyde added a comment - Adding interface RelDataTypeSystem. The constructor of RelDataTypeFactoryImpl now takes a RelDataTypeSystem. Most applications will use RelDataTypeSystem.DEFAULT if you like. Also adding a new connection property, "typeSystem", and adding a method FrameworkConfig.getTypeSystem().
        Show
        julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-optiq/commit/0fd4e3d9 .

          People

          • Assignee:
            julianhyde Julian Hyde
            Reporter:
            jpullokkaran Laljo John Pullokkaran
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development