Uploaded image for project: 'Apache Drill'
  1. Apache Drill
  2. DRILL-4448

Specification of Ordering (ASC, DESC) on a sort plan node uses Strings for construction, should also allow for use of the corresponding Calcite Enums

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      Small change to provide a cleaner interface when constructing sort configurations in tests. The current class mixes together two tasks, converting between the strings we chose to put in the plans (ASC, DESC) and the Calcite enums, as well as validating the allowed values. Calcite has several values that we do not currently use like STRICTLY_ASCENDING/DESCENDING and CLUSTERED. We can break these two tasks apart to allow for construction directly from the Enum, but still provide validation for only the drill allowed values.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              jaltekruse Jason Altekruse
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: