----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25341/ -----------------------------------------------------------
Review request for hive and Prasanth_J. Bugs: HIVE-7078 https://issues.apache.org/jira/browse/HIVE-7078 Repository: hive-git Description ------- Changes FileSinkOperator to use RecordUpdater in cases where an ACID write is being done. Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java d4e61d8 ql/src/java/org/apache/hadoop/hive/ql/io/AcidUtils.java f584926 ql/src/java/org/apache/hadoop/hive/ql/io/HiveFileFormatUtils.java c3a83d4 ql/src/java/org/apache/hadoop/hive/ql/plan/FileSinkDesc.java 301dde5 ql/src/test/org/apache/hadoop/hive/ql/exec/TestFileSinkOperator.java PRE-CREATION Diff: https://reviews.apache.org/r/25341/diff/ Testing ------- Added a new unit test TestFileSinkOperator that tests writing of standard (non-ACID) data via RecordWriter and acid data via RecordUpdater, in both partitioned and non-partitioned cases. Thanks, Alan Gates