----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35609/#review89036 -----------------------------------------------------------
Ship it! Ship It! - Hanifi Gunes On June 18, 2015, 3:14 p.m., abdelhakim deneche wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35609/ > ----------------------------------------------------------- > > (Updated June 18, 2015, 3:14 p.m.) > > > Review request for drill and Hanifi Gunes. > > > Bugs: DRILL-3243 > https://issues.apache.org/jira/browse/DRILL-3243 > > > Repository: drill-git > > > Description > ------- > > changed RepeatedVarCharOutput to display the column name as part of the > exception's message > changed CompliantTestRecordReader to throw a DATA_READ user exception > added new unit test to TestNewTextReader > > > Diffs > ----- > > common/src/main/java/org/apache/drill/common/exceptions/UserException.java > 6f28a2b > > common/src/main/java/org/apache/drill/common/exceptions/UserRemoteException.java > 1b3fa42 > > exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/CompliantTextRecordReader.java > 254e0d8 > > exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/text/compliant/RepeatedVarCharOutput.java > 40276f4 > > exec/java-exec/src/test/java/org/apache/drill/exec/store/text/TestNewTextReader.java > 76674f9 > > Diff: https://reviews.apache.org/r/35609/diff/ > > > Testing > ------- > > all unit tests are passing along with functional and tpch100 > > > Thanks, > > abdelhakim deneche > >
