-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19723/
-----------------------------------------------------------
(Updated March 29, 2014, 12:02 a.m.)
Review request for drill, Jacques Nadeau, Steven Phillips, and Venki Korukanti.
Changes
-------
Addressed review comment.
Bugs: DRILL-452
https://issues.apache.org/jira/browse/DRILL-452
Repository: drill-git
Description
-------
This patch provides the foundation for drill convert_to() and convert_from()
functions along with couple of implementations.
Diffs (updated)
-----
common/src/main/antlr3/org/apache/drill/common/expression/parser/ExprLexer.g
be2a3f2
common/src/main/antlr3/org/apache/drill/common/expression/parser/ExprParser.g
b60e2d6
common/src/main/java/org/apache/drill/common/expression/ConvertExpression.java
PRE-CREATION
common/src/main/java/org/apache/drill/common/expression/ExpressionStringBuilder.java
80fd41b
common/src/main/java/org/apache/drill/common/expression/FunctionCallFactory.java
c6c7074
common/src/main/java/org/apache/drill/common/expression/visitors/AbstractExprVisitor.java
727a3dc
common/src/main/java/org/apache/drill/common/expression/visitors/AggregateChecker.java
630d00a
common/src/main/java/org/apache/drill/common/expression/visitors/ConstantChecker.java
60e997f
common/src/main/java/org/apache/drill/common/expression/visitors/ExprVisitor.java
e199f21
common/src/main/java/org/apache/drill/common/expression/visitors/ExpressionValidator.java
96666bd
common/src/main/java/org/apache/drill/common/types/Types.java fd8991d
exec/java-exec/src/main/java/org/apache/drill/exec/compile/sig/ConstantExpressionIdentifier.java
e476897
exec/java-exec/src/main/java/org/apache/drill/exec/expr/EvaluationVisitor.java
8dfba76
exec/java-exec/src/main/java/org/apache/drill/exec/expr/ExpressionTreeMaterializer.java
f764d32
exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/conv/BigIntBEConvertFrom.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/conv/BigIntBEConvertTo.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/conv/IntBEConvertFrom.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/conv/IntBEConvertTo.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/store/hive/HiveRecordReader.java
5bd520d
exec/java-exec/src/main/java/org/apache/drill/exec/util/ConvertUtil.java
PRE-CREATION
exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/TestConvertFunctions.java
PRE-CREATION
exec/java-exec/src/test/resources/functions/conv/test_int_be.json
PRE-CREATION
Diff: https://reviews.apache.org/r/19723/diff/
Testing
-------
* Existing Drill unit test suit.
* New test case TestConvertFunctions.
Thanks,
Aditya Kishore
