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

Review request for hive, Ashutosh Chauhan and Mohammad Islam.


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


Repository: hive-git


Description
-------

Optionally implements cloning via Kyro. This improved performance and works 
with Java 7.

TestRCFile changes were made because the test was being flaky.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/ColumnInfo.java 44065ff 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Operator.java fa3ad8f 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java fd5a4f0 
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcStruct.java 2b1f0d8 
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcUnion.java 1ada1b8 
  ql/src/java/org/apache/hadoop/hive/ql/plan/ExprNodeDesc.java 475c564 
  ql/src/java/org/apache/hadoop/hive/ql/plan/MapWork.java b2c3b3f 
  ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFUnion.java 
04eb40b 
  ql/src/test/org/apache/hadoop/hive/ql/io/TestRCFile.java 62f1ab7 
  
ql/src/test/org/apache/hadoop/hive/serde2/CustomNonSettableListObjectInspector1.java
 867e030 
  
ql/src/test/org/apache/hadoop/hive/serde2/CustomNonSettableStructObjectInspector1.java
 52c82a5 
  serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyPrimitive.java ff277e0 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/LazyListObjectInspector.java
 75beca7 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/LazyMapObjectInspector.java
 7a30b68 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/LazySimpleStructObjectInspector.java
 08400f1 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/LazyUnionObjectInspector.java
 32f8403 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/AbstractPrimitiveLazyObjectInspector.java
 eef9c99 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyBinaryObjectInspector.java
 a68a1a1 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyStringObjectInspector.java
 25ab025 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryListObjectInspector.java
 89bc722 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryMapObjectInspector.java
 1cd4218 
  
serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/objectinspector/LazyBinaryStructObjectInspector.java
 7141aac 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ColumnarStructObjectInspector.java
 57ef071 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedListObjectInspector.java
 6283326 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedMapObjectInspector.java
 cce7fde 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedStructObjectInspector.java
 26f2511 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedUnionObjectInspector.java
 ecef3c7 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/MetadataListStructObjectInspector.java
 b8d3421 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ReflectionStructObjectInspector.java
 4689dc4 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/StandardConstantListObjectInspector.java
 5aad51f 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/StandardConstantMapObjectInspector.java
 ff190af 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/StandardListObjectInspector.java
 af14663 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/StandardMapObjectInspector.java
 2f5c1d9 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/StandardStructObjectInspector.java
 e89163c 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/StandardUnionObjectInspector.java
 21fa2f7 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/UnionStructObjectInspector.java
 6e3dfea 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/AbstractPrimitiveJavaObjectInspector.java
 59cb6ad 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/AbstractPrimitiveObjectInspector.java
 8620af3 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/AbstractPrimitiveWritableObjectInspector.java
 1e11875 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorUtils.java
 7b8f947 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantBinaryObjectInspector.java
 fe91a82 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantBooleanObjectInspector.java
 03a92e8 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantByteObjectInspector.java
 808f8e4 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantDateObjectInspector.java
 3bd87e9 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantDoubleObjectInspector.java
 4fe1639 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantFloatObjectInspector.java
 d40bdc7 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantHiveDecimalObjectInspector.java
 c0413e2 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantIntObjectInspector.java
 8cb8c1c 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantLongObjectInspector.java
 42f9eed 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantShortObjectInspector.java
 a476fa0 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantStringObjectInspector.java
 802357b 
  
serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableConstantTimestampObjectInspector.java
 2ddd3c3 

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


Testing
-------

Many unit tests pass. Pre-commit testing will do a full run.


Thanks,

Brock Noland

Reply via email to