You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Laszlo Bodor <bo...@gmail.com> on 2019/03/22 07:46:24 UTC

Review Request 70270: HIVE-21465 Introduce a base abstract VectorExpression to eliminate further code duplications

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

Review request for hive.


Repository: hive-git


Description
-------

HIVE-21465 Introduce a base abstract VectorExpression to eliminate further code duplications


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/BaseVectorExpression.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/CastDoubleToDecimal.java eb925af64c 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/CastLongToDecimal.java fa88e3ffb0 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/ConvertDecimal64ToDecimal.java 04a2cba62a 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/FuncDecimalToDouble.java 09215509de 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/FuncDoubleToDecimal.java 7fc8f6ab40 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/FuncLongToDecimal.java d3136c3ed6 
  ql/src/java/org/apache/hadoop/hive/ql/exec/vector/expressions/FuncLongToString.java 5805afcf23 


Diff: https://reviews.apache.org/r/70270/diff/1/


Testing
-------

vectorization unit + qtests passed


Thanks,

Laszlo Bodor