hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Navis Ryu" <navis....@nexr.com>
Subject Re: Review Request 29037: Avoid creating serde for partitions if possible in FetchTask
Date Thu, 18 Dec 2014 07:04:38 GMT

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

(Updated Dec. 18, 2014, 7:04 a.m.)


Review request for hive.


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


Repository: hive-git


Description
-------

Currently, FetchTask creates SerDe instance thrice for each partition, which can be avoided
if it's same with table SerDe.


Diffs (updated)
-----

  data/files/datatypes.txt 0228a27 
  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 1355230 
  ql/src/java/org/apache/hadoop/hive/ql/exec/FetchOperator.java 6338c3c 
  ql/src/java/org/apache/hadoop/hive/ql/exec/PartitionKeySampler.java 5d126a5 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/ExecMapper.java ca65a8e 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 29be691 
  ql/src/java/org/apache/hadoop/hive/ql/plan/FetchWork.java 32d84ea 
  ql/src/java/org/apache/hadoop/hive/ql/plan/TableDesc.java 374e8b6 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedListObjectInspector.java
6a9215b 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedMapObjectInspector.java
975d5cd 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedObjectInspectorFactory.java
2db3819 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedStructObjectInspector.java
ef66e97 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/DelegatedUnionObjectInspector.java
521fdd6 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorConverters.java
8a42577 
  serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorConverter.java
49884b8 

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


Testing
-------


Thanks,

Navis Ryu


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