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