-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35609/
-----------------------------------------------------------

(Updated June 25, 2015, 3:52 p.m.)


Review request for drill and Hanifi Gunes.


Changes
-------

rebased on top of master


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 (updated)
-----

  common/src/main/java/org/apache/drill/common/exceptions/UserException.java 
13c17bd 
  
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

Reply via email to