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

Make RelCollation trait and AbstractRelNode.getCollationList consistent

    Details

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

      Description

      Currently getCollationList is not consistent with RelCollation trait since a node can have only one collation trait, thus you cannot express the node that is sorted on multiple collations at the same time.

      We should either drop getCollationList in favour of getCollation or make collation trait support multiple "collation lists".

        Issue Links

          Activity

          Hide
          julianhyde Julian Hyde added a comment -

          I'm solving these very issues as part of CALCITE-526. If you do a merge join of emp and dept both sorted on deptno, the output contain two deptno columns and will be sorted on both.

          Show
          julianhyde Julian Hyde added a comment - I'm solving these very issues as part of CALCITE-526 . If you do a merge join of emp and dept both sorted on deptno, the output contain two deptno columns and will be sorted on both.
          Show
          julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/2709896e .
          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:
              julianhyde Julian Hyde
              Reporter:
              vladimirsitnikov Vladimir Sitnikov
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development