You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@drill.apache.org by "Venki Korukanti (JIRA)" <ji...@apache.org> on 2015/04/23 20:40:38 UTC

[jira] [Created] (DRILL-2858) Refactor hash expression construction in InsertLocalExchangeVisitor and PrelUtil into one place

Venki Korukanti created DRILL-2858:
--------------------------------------

             Summary: Refactor hash expression construction in InsertLocalExchangeVisitor and PrelUtil into one place
                 Key: DRILL-2858
                 URL: https://issues.apache.org/jira/browse/DRILL-2858
             Project: Apache Drill
          Issue Type: Bug
            Reporter: Venki Korukanti
            Assignee: Venki Korukanti


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.





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)