----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35808/#review89433 -----------------------------------------------------------
+ minor issues + suggestions: formatting, finals exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 108) <https://reviews.apache.org/r/35808/#comment142042> final exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 118) <https://reviews.apache.org/r/35808/#comment142043> Also, this format: UserException.parseError() .message("...",functionName) .build(logger); exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 180) <https://reviews.apache.org/r/35808/#comment142044> static method? Also, final argument exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 184) <https://reviews.apache.org/r/35808/#comment142045> same formatting issue as above exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 191) <https://reviews.apache.org/r/35808/#comment142046> static method? final arguments exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 193) <https://reviews.apache.org/r/35808/#comment142063> typeName.size()? exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 194) <https://reviews.apache.org/r/35808/#comment142051> final exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 197) <https://reviews.apache.org/r/35808/#comment142047> typename.size()? exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 198) <https://reviews.apache.org/r/35808/#comment142048> final exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 199) <https://reviews.apache.org/r/35808/#comment142049> final exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java (line 205) <https://reviews.apache.org/r/35808/#comment142050> new StringBuilder(...) .append(...) .append(...); exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 29) <https://reviews.apache.org/r/35808/#comment142052> final arguments exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 30) <https://reviews.apache.org/r/35808/#comment142053> finals exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 52) <https://reviews.apache.org/r/35808/#comment142054> final (here and below) exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 64) <https://reviews.apache.org/r/35808/#comment142055> final exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 73) <https://reviews.apache.org/r/35808/#comment142056> final arguments exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 75) <https://reviews.apache.org/r/35808/#comment142057> final exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 76) <https://reviews.apache.org/r/35808/#comment142058> final exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 77) <https://reviews.apache.org/r/35808/#comment142059> final exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 80) <https://reviews.apache.org/r/35808/#comment142060> final exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java (line 81) <https://reviews.apache.org/r/35808/#comment142062> final String bestMatch = namesToSearch.get(minIndex); - Sudheesh Katkam On June 24, 2015, 4:57 p.m., Parth Chandra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35808/ > ----------------------------------------------------------- > > (Updated June 24, 2015, 4:57 p.m.) > > > Review request for drill, Mehant Baid, Sudheesh Katkam, and Venki Korukanti. > > > Repository: drill-git > > > Description > ------- > > DRILL-2862: Convert_to/Convert_From throw assertion when an incorrect > encoding type is specified or if the encoding type is not a string literal. > > Instead of an assertion when user input is wrong, we now throw an exception > with the appropriate error message. > For the case where the user types in a type name incorrectly, the error > message also provides a helpful suggestion. The suggested name is selected > from the list of available functions. > > For example: > > select convert_from(foo, 'UTF') from dfs.`/table_foo` > > will print the following error: > > Error: UNSUPPORTED_OPERATION ERROR: CONVERT_FROM does not support > conversion from type 'UTF'. > Did you mean UTF8? > [Error Id: 87ed2941-f9c2-4c35-8ff2-a3f21eae1104 on localhost:31010] > (state=,code=0) > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/PreProcessLogicalRel.java > 0f8e45a > > exec/java-exec/src/main/java/org/apache/drill/exec/util/ApproximateStringMatcher.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/35808/diff/ > > > Testing > ------- > > All regression tests > > > Thanks, > > Parth Chandra > >
