> On Sept. 21, 2014, 6:09 p.m., Jacques Nadeau wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/holders/ValueHolder.java, > > line 24 > > <https://reviews.apache.org/r/25877/diff/2/?file=699498#file699498line24> > > > > You can't add any methods to ValueHolder or its subclasses since we're > > trying to do scalar replacement.
I thought it would be ok to add a method just to return a static data field, even when we try to use scalar replacement. Or, how about change getType() into a static method? Will a static method affect scalar replacement? > On Sept. 21, 2014, 6:09 p.m., Jacques Nadeau wrote: > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/interpreter/InterpreterEvalator.java, > > line 40 > > <https://reviews.apache.org/r/25877/diff/2/?file=699493#file699493line40> > > > > Misspelling Will change the spelling. - Jinfeng ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25877/#review54102 ----------------------------------------------------------- On Sept. 21, 2014, 4:42 p.m., Jinfeng Ni wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25877/ > ----------------------------------------------------------- > > (Updated Sept. 21, 2014, 4:42 p.m.) > > > Review request for drill and Steven Phillips. > > > Repository: drill-git > > > Description > ------- > > This patch provides the interpreted model to evaluate a Drill expression, > consists of constants, DrillSimpleFunction, Case Expression, Boolean > operator. > > It does not switch the existing Drill's execution framework to this new > model. As the next step of work, we will modify the existing > ProjectRecordBatch / FilterRecordBatch, such that the projected expression, > or filter expression would use this interpreted model, when necessary. > > In this patch, the only way to use this new interpreted model is to through > unit test ( see ExpressionInterpreterTest.java for example). > > We add a new mvn module under drill_root/exec. The new module is the place > where all the static function interpreters will be generated into, and > compiled, during maven build process. > > > Diffs > ----- > > exec/interpreter/pom.xml PRE-CREATION > > exec/interpreter/src/test/java/org/apache/drill/exec/expr/ExpressionInterpreterTest.java > PRE-CREATION > exec/java-exec/src/main/codegen/templates/TypeHelper.java 9bb2507 > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillBooleanOPHolder.java > 9032d37 > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillFuncHolder.java > 1852731 > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/DrillSimpleFuncHolder.java > 4731200 > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/FunctionConverter.java > 402a6fc > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/interpreter/DrillFuncInterpreter.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/interpreter/DrillSimpleFuncInterpreter.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/interpreter/InterpreterBuilder.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/interpreter/InterpreterEvalator.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/interpreter/InterpreterGenerator.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/holders/ComplexHolder.java > e1025df > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/holders/RepeatedListHolder.java > 09746da > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/holders/RepeatedMapHolder.java > 247f75e > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/holders/ValueHolder.java > 5c2adc6 > > exec/java-exec/src/main/java/org/apache/drill/exec/vector/ValueHolderHelper.java > 31afb6f > exec/pom.xml fe2a75e > > Diff: https://reviews.apache.org/r/25877/diff/ > > > Testing > ------- > > 1. Run existing JUnit test suite. > > 2. Add a new ExpressionInterpreterTest.java. > > > Thanks, > > Jinfeng Ni > >
