> On June 17, 2015, 9:54 p.m., Hanifi Gunes wrote: > > exec/java-exec/src/main/codegen/templates/TypeHelper.java, line 474 > > <https://reviews.apache.org/r/35573/diff/1/?file=986539#file986539line474> > > > > Would you use buildErrMsg here as well?
I originally didn't want to change these as they already display a "good enough" error message. I will make the change and update the patch - abdelhakim ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35573/#review88285 ----------------------------------------------------------- On June 17, 2015, 8:30 p.m., abdelhakim deneche wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35573/ > ----------------------------------------------------------- > > (Updated June 17, 2015, 8:30 p.m.) > > > Review request for drill and Hanifi Gunes. > > > Bugs: DRILL-3304 > https://issues.apache.org/jira/browse/DRILL-3304 > > > Repository: drill-git > > > Description > ------- > > Made some changes to TypeHelper template to display the failed "operation" > and minor-type + data-mode when an UnsupportedOperationException is thrown > > > Diffs > ----- > > common/src/main/java/org/apache/drill/common/exceptions/ErrorHelper.java > 5dd9b67 > exec/java-exec/src/main/codegen/templates/TypeHelper.java ad818bd > > Diff: https://reviews.apache.org/r/35573/diff/ > > > Testing > ------- > > all unit tests are passing along with functional/tpch100 > > > Thanks, > > abdelhakim deneche > >
