| Age | Commit message (Collapse) | Author |
|---|
|
closes #1207
|
|
Integrate Jacques's interpreter based partition pruning with Jason's interpreter refactoring that removed interpreter module and added that functionality in the exec module.
Ensure boolean operators are correctly handled when traversing expression tree to find partition filters.
Resolve merge conflicts after rebasing to master branch.
Additional fixes for handling OR conditions.
|
|
|
|
expression interpretation.
Changed interpreted evaluation to run the setup method after the input parameters have been set using reflection
so they are available in the case where the inputs are constant and are used in the setup method.
Changes that were originally committed in later patches for 2060 and 2173, as they are needed for this to run on its own:
- Change to DrillSimpleFuncHolder that belongs with the interpreter refactoring.
- ValueHolderHelper changes needed for interpreter refactoring.
Updates after review comments from Jinfeng:
Change the DrillSimpleFunc creation in the new interpreter to use a direct reference to the class type of the Function rather than the class name.
Add test case for 'like' function evaluated in the interpreter (has a meaningful setup method that uses one of the inputs to initialize a pattern matcher)
Update from Aman's review:
Add a test case for using a cast in interpreted expression evaluation.
|
|
fill in holes from removing RecordBatch from UDFs - Move list of accepted injectable types into the UdfUtilities interface for easier maintenance.
Fix one more new function to remove RecordBatch from setup method.
Add back UdfUtilities interface to FragmentContext lost in rebasing.
Remove unneeded testcase in TestExampleQueries, remove commented out code in InterpreterGenerator
previously used to handle DrillBuf injectables, now being handled by reflection to set the DrillBuf
rather than the previously generated code that would use the direct reference to the incoming
RecordBatch to get a buffer.
Fix docs on UdfUtilities and package docs for drill/exec/expr, belongs with 2143 part 2 patch.
Fix belongs with 2143, change interpreter to output a ValueHolder instead of a ValueVector in the case of a constant expression.
2143 update - Use reflection to remove boilerplate for adding new injectable types for UDFs.
|
|
|
|
|
|
|
|
|
|
The following pull requests have been either been merged or closed as of this release.
closes #1 (via 207f2e4)
closes #3 (*No longer relevant*)
closes #8 (via cb3132a)
closes #11 (via 727adb7, a73512d)
closes #12 (*Issue fixed via DRILL-88*)
closes #13 (*Closed as Won't Fix*)
closes #14 (via f0be80d)
closes #17 (via a38856c)
closes #35 (via 741aab0)
closes #60 (via 1648195)
closes #64 (Resolved as part of other function merges)
closes #65 (via 28dd76a)
closes #66 (via 4862b2b)
closes #67 (via 2ca9c90)
|
|
Fix OOM while running InterpreterBuilder
|
