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

Refactor hash expression construction in InsertLocalExchangeVisitor and PrelUtil into one place

    XMLWordPrintableJSON

    Details

      Description

      Currently there are two place where we construct the hash expression based on the partition fields:
      1. InsertLocalExchangeVistor (generates RexExpr type)
      2. PRelUtil.getHashExpression (generate LogicalExpression type)

      Having this logic in two places makes them prone to errors and they can easily go out of sync causing hard to debug verification failures.

        Attachments

        1. DRILL-2858-1.patch
          25 kB
          Venki Korukanti

          Activity

            People

            • Assignee:
              vkorukanti Venki Korukanti
              Reporter:
              vkorukanti Venki Korukanti
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: