You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Navis Ryu <na...@nexr.com> on 2014/11/20 02:44:46 UTC

Review Request 28274: Refactoring of PassThroughOutputFormat

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

Review request for hive.


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


Repository: hive-git


Description
-------

It's overly complicated just for doing simple wrapping of output format. Before things get more worse, we should refactor this codes.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 4fb30bc 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java 913288f 
  ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/RowContainer.java 5271e91 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveFileFormatUtils.java 2ac60c0 
  ql/src/java/org/apache/hadoop/hive/ql/io/HivePassThroughOutputFormat.java 04eff93 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Partition.java 13277a9 
  ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java 211ab6c 
  ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java 4891518 
  ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 016a6d8 
  ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableDesc.java deba198 
  ql/src/java/org/apache/hadoop/hive/ql/plan/PartitionDesc.java 10c38d9 
  ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java fdc1f62 
  ql/src/java/org/apache/hadoop/hive/ql/plan/TableDesc.java 408ddf5 

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


Testing
-------


Thanks,

Navis Ryu