hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Remus Rusanu" <rem...@microsoft.com>
Subject Review Request 17737: Add DECIMAL support to vectorized group by operator
Date Wed, 05 Feb 2014 11:04:39 GMT

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

Review request for hive, Eric Hanson and Jitendra Pandey.


Bugs: HIVE-6344
    https://issues.apache.org/jira/browse/HIVE-6344


Repository: hive-git


Description
-------

Implements Decimal aggregate operators, decimal vector hash key wrapper, extends vectorizer
to support decimal in GBY.


Diffs
-----

  ant/src/org/apache/hadoop/hive/ant/GenVectorCode.java 1b76fc9 
  common/src/java/org/apache/hadoop/hive/common/type/Decimal128.java 2e0f058 
  common/src/java/org/apache/hadoop/hive/common/type/HiveDecimal.java 29c5168 
  common/src/java/org/apache/hadoop/hive/common/type/UnsignedInt128.java fb3c346 
  common/src/java/org/apache/hive/common/util/Decimal128FastBuffer.java PRE-CREATION 
  ql/src/gen/vectorization/UDAFTemplates/VectorUDAFMinMaxDecimal.txt PRE-CREATION 
  ql/src/gen/vectorization/UDAFTemplates/VectorUDAFVarDecimal.txt PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorHashKeyWrapper.java f083d86 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorHashKeyWrapperBatch.java e978110

  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizationContext.java f5ab731 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/VectorizedRowBatchCtx.java f513188 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorExpressionWriter.java
e5c3aa4 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/VectorExpressionWriterFactory.java
a242fef 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/aggregates/VectorUDAFAvgDecimal.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/aggregates/VectorUDAFSumDecimal.java
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/Vectorizer.java ad96fa5 
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFAverage.java 1a00800 
  ql/src/test/org/apache/hadoop/hive/ql/exec/vector/TestVectorGroupByOperator.java a2b45f8

  ql/src/test/org/apache/hadoop/hive/ql/exec/vector/util/FakeVectorRowBatchFromObjectIterables.java
c8eaea1 
  serde/src/test/org/apache/hadoop/hive/serde2/io/TestHiveDecimalWritable.java PRE-CREATION


Diff: https://reviews.apache.org/r/17737/diff/


Testing
-------

New unit tests added, manually tested decimal GBY queries


Thanks,

Remus Rusanu


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message