----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33662/#review82541 -----------------------------------------------------------
Ship it! Ship It! - Mehant Baid On May 4, 2015, 10:09 p.m., Venki Korukanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/33662/ > ----------------------------------------------------------- > > (Updated May 4, 2015, 10:09 p.m.) > > > Review request for drill and Mehant Baid. > > > Repository: drill-git > > > Description > ------- > > Please see https://issues.apache.org/jira/browse/DRILL-2902 for details. > > Apart from adding new UDFs, also refactored the context information stored in > PlanFragment into a separate message. Refactored QueryDateTimeInfo into > ContextInformation to provide one interface for all query context replated > info. > > > Diffs > ----- > > exec/java-exec/src/main/java/org/apache/drill/exec/client/DrillClient.java > 4576eb4 > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/ContextFunctions.java > PRE-CREATION > > exec/java-exec/src/main/java/org/apache/drill/exec/expr/fn/impl/DateTypeFunctions.java > 9c932d6 > > exec/java-exec/src/main/java/org/apache/drill/exec/ops/ContextInformation.java > PRE-CREATION > exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java > 09a7568 > exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryContext.java > 6414f56 > > exec/java-exec/src/main/java/org/apache/drill/exec/ops/QueryDateTimeInfo.java > f3cc666 > exec/java-exec/src/main/java/org/apache/drill/exec/ops/UdfUtilities.java > 1cdece1 > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/SimpleParallelizer.java > 66ba229 > > exec/java-exec/src/main/java/org/apache/drill/exec/rpc/user/UserSession.java > 527bac0 > exec/java-exec/src/main/java/org/apache/drill/exec/util/Utilities.java > 8efb9e7 > > exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java > 4249cbe > > exec/java-exec/src/test/java/org/apache/drill/exec/fn/impl/TestContextFunctions.java > PRE-CREATION > > exec/java-exec/src/test/java/org/apache/drill/exec/fn/interp/ExpressionInterpreterTest.java > 04e1980 > > exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/TestLocalExchange.java > 9758eb0 > > exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/partitionsender/TestPartitionSender.java > 6a6a7e0 > > exec/java-exec/src/test/java/org/apache/drill/exec/pop/TestFragmentChecker.java > 32e3bf9 > > exec/java-exec/src/test/java/org/apache/drill/exec/rpc/user/security/TestCustomUserAuthenticator.java > 70d43b6 > > exec/java-exec/src/test/java/org/apache/drill/exec/testing/TestExceptionInjection.java > 604f375 > > exec/java-exec/src/test/java/org/apache/drill/exec/testing/TestPauseInjection.java > 508b10c > protocol/src/main/java/org/apache/drill/exec/proto/BitControl.java 813d961 > protocol/src/main/java/org/apache/drill/exec/proto/SchemaBitControl.java > 5e7562e > protocol/src/main/java/org/apache/drill/exec/proto/beans/PlanFragment.java > f6fbce1 > > protocol/src/main/java/org/apache/drill/exec/proto/beans/QueryContextInformation.java > PRE-CREATION > protocol/src/main/protobuf/BitControl.proto 0424725 > > Diff: https://reviews.apache.org/r/33662/diff/ > > > Testing > ------- > > Added unittests to test the new UDFs added in the patch. > > > Thanks, > > Venki Korukanti > >