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

            depends upon

            Bug - A problem which impairs or prevents the functions of the product. CALCITE-526 Add EnumerableMergeJoin, which exploits sorted inputs

            • Major - Major loss of function.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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.
              Hide
              Permalink
              julianhyde Julian Hyde added a comment -

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

              Show
              julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/incubator-calcite/commit/2709896e .
              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:
                  julianhyde Julian Hyde
                  Reporter:
                  vladimirsitnikov Vladimir Sitnikov
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:

                    Development