On Dec. 2, 2013, 10:55 p.m., Sergey Shelukhin wrote: > > I did not see the junit test. I also see trailing white space in template > > files still. Did you upload latest diff to ReviewBoard? > >
not yet, working on q file - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15869/#review29625 ----------------------------------------------------------- On Nov. 27, 2013, 2:20 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15869/ > ----------------------------------------------------------- > > (Updated Nov. 27, 2013, 2:20 a.m.) > > > Review request for hive. > > > Repository: hive-git > > > Description > ------- > > HIVE-5895 vectorization handles division by zero differently from normal > execution > > > Diffs > ----- > > ant/src/org/apache/hadoop/hive/ant/GenVectorCode.java 5dc50d6 > ql/src/gen/vectorization/ExpressionTemplates/ColumnDivideColumn.txt > PRE-CREATION > ql/src/gen/vectorization/ExpressionTemplates/ColumnDivideScalar.txt > PRE-CREATION > ql/src/gen/vectorization/ExpressionTemplates/ScalarDivideColumn.txt > PRE-CREATION > > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/LongColDivideLongColumn.java > 482df3c > > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/LongColDivideLongScalar.java > 7d517ca > > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/LongScalarDivideLongColumn.java > e1ecefa > ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/NullUtil.java > e1867a2 > ql/src/test/queries/clientpositive/vectorization_div0.q PRE-CREATION > ql/src/test/results/clientpositive/vectorization_div0.q.out PRE-CREATION > ql/src/test/results/clientpositive/vectorization_short_regress.q.out > c9296e1 > > Diff: https://reviews.apache.org/r/15869/diff/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >
