----------------------------------------------------------- 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