----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17005/#review33024 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/io/orc/RecordReaderImpl.java <https://reviews.apache.org/r/17005/#comment62175> Please change updateBigInteger() name to update() for consistency. ql/src/java/org/apache/hadoop/hive/ql/io/orc/RecordReaderImpl.java <https://reviews.apache.org/r/17005/#comment62176> Please add a comment to explain why you are conditionally changing the scale. - Eric Hanson On Jan. 27, 2014, 11:20 p.m., Jitendra Pandey wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17005/ > ----------------------------------------------------------- > > (Updated Jan. 27, 2014, 11:20 p.m.) > > > Review request for hive and Eric Hanson. > > > Bugs: HIVE-6178 > https://issues.apache.org/jira/browse/HIVE-6178 > > > Repository: hive-git > > > Description > ------- > > vectorized reader for DECIMAL datatype for ORC format. > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/common/type/Decimal128.java 15025cf > common/src/java/org/apache/hadoop/hive/common/type/UnsignedInt128.java > d71ebb3 > common/src/test/org/apache/hadoop/hive/common/type/TestUnsignedInt128.java > fbb2aa0 > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/DecimalColumnVector.java > 23564bb > ql/src/java/org/apache/hadoop/hive/ql/io/orc/RecordReaderImpl.java 0df82b9 > ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestVectorizedORCReader.java > 0d5b7ff > > Diff: https://reviews.apache.org/r/17005/diff/ > > > Testing > ------- > > > Thanks, > > Jitendra Pandey > >