drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Venki Korukanti (JIRA)" <j...@apache.org>
Subject [jira] [Created] (DRILL-2858) Refactor hash expression construction in InsertLocalExchangeVisitor and PrelUtil into one place
Date Thu, 23 Apr 2015 18:40:38 GMT
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)

Mime
View raw message