-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20743/
-----------------------------------------------------------
Review request for drill.
Bugs: DRILL-516
https://issues.apache.org/jira/browse/DRILL-516
Repository: drill-git
Description
-------
Optiq rewrites the extract function as div, mod based on the data types. Since
Drill does not know about the data type we cannot rewrite the extract function.
This patch passes a custom convertlet to Optiq which would treat extract as a
normal function and not rewrite it.
Diffs
-----
exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillOptiq.java
f46f012
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillConvertletTable.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillExtractConvertlet.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/DrillSqlWorker.java
8892a8f
exec/java-exec/src/test/java/org/apache/drill/TestTpchPlanning.java 7327c67
Diff: https://reviews.apache.org/r/20743/diff/
Testing
-------
Manual testing using sqlline.
Added tpch07 test back in TestTpchPlanning.
Couldn't add tpch08, tpch09 back because they are now hitting "cannot plan
exception".
Thanks,
Mehant Baid