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

Review request for hive, cheng xu and Dong Chen.


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


Repository: hive-git


Description
-------

This patch allocates only selected schema objects in HiveStructConverter class 
in order to reduce memory usage.
It also removes some unused imports from the files that were edited for this 
patch.

As I saw on the code for HiveStructConverter, the class was allocating enough 
objects as the number of fields the table schema has. But these objects weren't 
used anywhere on the code.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/FetchTask.java 
c735be168e2f6f10c30cd1c3f2b46aad5260256f 
  ql/src/java/org/apache/hadoop/hive/ql/exec/SMBMapJoinOperator.java 
2c9e81fcffe49f803ccb41bb2b6ce74f230b2aca 
  ql/src/java/org/apache/hadoop/hive/ql/exec/TableScanOperator.java 
cb010fb9b680ed1b95cc3efc4e1b806ce7ac6383 
  ql/src/java/org/apache/hadoop/hive/ql/exec/mr/MapredLocalTask.java 
9f3df991d6e8f450cdada28ddb03449aac897487 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveInputFormat.java 
9d5730d3d5adefb65117f4c76aacd29426e380aa 
  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/DataWritableRecordConverter.java
 a43661eb54ba29692c07c264584b5aecf648ef99 
  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/convert/HiveStructConverter.java
 f95d15eddc21bc432fa53572de5756751a13341a 
  
ql/src/java/org/apache/hadoop/hive/ql/io/parquet/read/DataWritableReadSupport.java
 47cd68200d3be9260aa35385d0dade74d7dc215d 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/serde/ParquetHiveSerDe.java 
7fd5e9612d4e3c9bf3b816bc48dbdbe59fb8a5a8 
  ql/src/java/org/apache/hadoop/hive/ql/optimizer/ColumnPrunerProcFactory.java 
f49b7cd931466d9aefd7c84aa37e2483ca1aa5de 
  ql/src/java/org/apache/hadoop/hive/ql/plan/TableScanDesc.java 
0e85990f5abc4aa3072ed53f8397e858f5b4062d 
  serde/src/java/org/apache/hadoop/hive/serde2/ColumnProjectionUtils.java 
e403ad9b34c482bf720bdec1c91057a5f53d216b 

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


Testing
-------


Thanks,

Sergio Pena

Reply via email to