You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hive.apache.org by Ashutosh Chauhan <ha...@apache.org> on 2014/04/23 04:06:43 UTC

Review Request 20598: Make Hive Output Format backward compatible.

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

Review request for hive and Prasanth_J.


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


Repository: hive-git


Description
-------

Make Hive Output Format backward compatible.


Diffs
-----

  contrib/src/java/org/apache/hadoop/hive/contrib/fileformat/base64/Base64TextOutputFormat.java bc601c4 
  hbase-handler/src/java/org/apache/hadoop/hive/hbase/HiveHFileOutputFormat.java 6d383b5 
  itests/util/src/main/java/org/apache/hadoop/hive/ql/io/udf/Rot13OutputFormat.java 40927f6 
  ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 6af6b2d 
  ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java c52a093 
  ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/PTFRowContainer.java 4ab5516 
  ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/RowContainer.java 768467e 
  ql/src/java/org/apache/hadoop/hive/ql/io/AcidOutputFormat.java 551f3aa 
  ql/src/java/org/apache/hadoop/hive/ql/io/FSRecordWriter.java 83ac010 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveBinaryOutputFormat.java 6768292 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveFileFormatUtils.java 95db96b 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveIgnoreKeyTextOutputFormat.java ad6e4ba 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveNullValueSequenceFileOutputFormat.java ef6a982 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveOutputFormat.java e5ac805 
  ql/src/java/org/apache/hadoop/hive/ql/io/HivePassThroughOutputFormat.java 1fb5898 
  ql/src/java/org/apache/hadoop/hive/ql/io/HivePassThroughRecordWriter.java 2186944 
  ql/src/java/org/apache/hadoop/hive/ql/io/HiveSequenceFileOutputFormat.java 0cf00e9 
  ql/src/java/org/apache/hadoop/hive/ql/io/RCFileOutputFormat.java e21f9fe 
  ql/src/java/org/apache/hadoop/hive/ql/io/StatsProvidingRecordWriter.java PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/io/avro/AvroContainerOutputFormat.java 8d75b44 
  ql/src/java/org/apache/hadoop/hive/ql/io/avro/AvroGenericRecordWriter.java 73e1cdd 
  ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcOutputFormat.java 46c3bcc 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/MapredParquetOutputFormat.java b87c673 
  ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/ParquetRecordWriterWrapper.java cd603c2 
  ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java de4867a 
  ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestInputOutputFormat.java 5664f3f 
  ql/src/test/org/apache/hadoop/hive/ql/txn/compactor/CompactorTest.java eaabc71 

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


Testing
-------


Thanks,

Ashutosh Chauhan


Re: Review Request 20598: Make Hive Output Format backward compatible.

Posted by j....@gmail.com.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20598/#review41114
-----------------------------------------------------------

Ship it!


Ship It!

- Prasanth_J


On April 23, 2014, 2:06 a.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20598/
> -----------------------------------------------------------
> 
> (Updated April 23, 2014, 2:06 a.m.)
> 
> 
> Review request for hive and Prasanth_J.
> 
> 
> Bugs: HIVE-6952
>     https://issues.apache.org/jira/browse/HIVE-6952
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Make Hive Output Format backward compatible.
> 
> 
> Diffs
> -----
> 
>   contrib/src/java/org/apache/hadoop/hive/contrib/fileformat/base64/Base64TextOutputFormat.java bc601c4 
>   hbase-handler/src/java/org/apache/hadoop/hive/hbase/HiveHFileOutputFormat.java 6d383b5 
>   itests/util/src/main/java/org/apache/hadoop/hive/ql/io/udf/Rot13OutputFormat.java 40927f6 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 6af6b2d 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java c52a093 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/PTFRowContainer.java 4ab5516 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/RowContainer.java 768467e 
>   ql/src/java/org/apache/hadoop/hive/ql/io/AcidOutputFormat.java 551f3aa 
>   ql/src/java/org/apache/hadoop/hive/ql/io/FSRecordWriter.java 83ac010 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveBinaryOutputFormat.java 6768292 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveFileFormatUtils.java 95db96b 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveIgnoreKeyTextOutputFormat.java ad6e4ba 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveNullValueSequenceFileOutputFormat.java ef6a982 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveOutputFormat.java e5ac805 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HivePassThroughOutputFormat.java 1fb5898 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HivePassThroughRecordWriter.java 2186944 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveSequenceFileOutputFormat.java 0cf00e9 
>   ql/src/java/org/apache/hadoop/hive/ql/io/RCFileOutputFormat.java e21f9fe 
>   ql/src/java/org/apache/hadoop/hive/ql/io/StatsProvidingRecordWriter.java PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/io/avro/AvroContainerOutputFormat.java 8d75b44 
>   ql/src/java/org/apache/hadoop/hive/ql/io/avro/AvroGenericRecordWriter.java 73e1cdd 
>   ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcOutputFormat.java 46c3bcc 
>   ql/src/java/org/apache/hadoop/hive/ql/io/parquet/MapredParquetOutputFormat.java b87c673 
>   ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/ParquetRecordWriterWrapper.java cd603c2 
>   ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java de4867a 
>   ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestInputOutputFormat.java 5664f3f 
>   ql/src/test/org/apache/hadoop/hive/ql/txn/compactor/CompactorTest.java eaabc71 
> 
> Diff: https://reviews.apache.org/r/20598/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>


Re: Review Request 20598: Make Hive Output Format backward compatible.

Posted by j....@gmail.com.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20598/#review41113
-----------------------------------------------------------

Ship it!


LGTM.

- Prasanth_J


On April 23, 2014, 2:06 a.m., Ashutosh Chauhan wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/20598/
> -----------------------------------------------------------
> 
> (Updated April 23, 2014, 2:06 a.m.)
> 
> 
> Review request for hive and Prasanth_J.
> 
> 
> Bugs: HIVE-6952
>     https://issues.apache.org/jira/browse/HIVE-6952
> 
> 
> Repository: hive-git
> 
> 
> Description
> -------
> 
> Make Hive Output Format backward compatible.
> 
> 
> Diffs
> -----
> 
>   contrib/src/java/org/apache/hadoop/hive/contrib/fileformat/base64/Base64TextOutputFormat.java bc601c4 
>   hbase-handler/src/java/org/apache/hadoop/hive/hbase/HiveHFileOutputFormat.java 6d383b5 
>   itests/util/src/main/java/org/apache/hadoop/hive/ql/io/udf/Rot13OutputFormat.java 40927f6 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/FileSinkOperator.java 6af6b2d 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java c52a093 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/PTFRowContainer.java 4ab5516 
>   ql/src/java/org/apache/hadoop/hive/ql/exec/persistence/RowContainer.java 768467e 
>   ql/src/java/org/apache/hadoop/hive/ql/io/AcidOutputFormat.java 551f3aa 
>   ql/src/java/org/apache/hadoop/hive/ql/io/FSRecordWriter.java 83ac010 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveBinaryOutputFormat.java 6768292 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveFileFormatUtils.java 95db96b 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveIgnoreKeyTextOutputFormat.java ad6e4ba 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveNullValueSequenceFileOutputFormat.java ef6a982 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveOutputFormat.java e5ac805 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HivePassThroughOutputFormat.java 1fb5898 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HivePassThroughRecordWriter.java 2186944 
>   ql/src/java/org/apache/hadoop/hive/ql/io/HiveSequenceFileOutputFormat.java 0cf00e9 
>   ql/src/java/org/apache/hadoop/hive/ql/io/RCFileOutputFormat.java e21f9fe 
>   ql/src/java/org/apache/hadoop/hive/ql/io/StatsProvidingRecordWriter.java PRE-CREATION 
>   ql/src/java/org/apache/hadoop/hive/ql/io/avro/AvroContainerOutputFormat.java 8d75b44 
>   ql/src/java/org/apache/hadoop/hive/ql/io/avro/AvroGenericRecordWriter.java 73e1cdd 
>   ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcOutputFormat.java 46c3bcc 
>   ql/src/java/org/apache/hadoop/hive/ql/io/parquet/MapredParquetOutputFormat.java b87c673 
>   ql/src/java/org/apache/hadoop/hive/ql/io/parquet/write/ParquetRecordWriterWrapper.java cd603c2 
>   ql/src/java/org/apache/hadoop/hive/ql/txn/compactor/CompactorMR.java de4867a 
>   ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestInputOutputFormat.java 5664f3f 
>   ql/src/test/org/apache/hadoop/hive/ql/txn/compactor/CompactorTest.java eaabc71 
> 
> Diff: https://reviews.apache.org/r/20598/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashutosh Chauhan
> 
>