hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Navis Ryu" <navis....@nexr.com>
Subject Review Request 28274: Refactoring of PassThroughOutputFormat
Date Thu, 20 Nov 2014 01:44:46 GMT

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


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