[ 
https://issues.apache.org/jira/browse/HIVE-7986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14122209#comment-14122209
 ] 

Julien Le Dem commented on HIVE-7986:
-------------------------------------

Actually here is a proposal on how this should work once the next release of 
Parquet is out:
https://github.com/julienledem/incubator-parquet-mr/commit/ccdd08c7030d9d4579b6beef2b93981e327637c5
I added a deprecated constructor in the meantime for compatibility.

> Hive depends on ParquetInputSplit constructor
> ---------------------------------------------
>
>                 Key: HIVE-7986
>                 URL: https://issues.apache.org/jira/browse/HIVE-7986
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Julien Le Dem
>
> As Parquet must be able to change the split implementation without breaking 
> Hive. (Users might want to use the latest Parquet with their current version 
> of Hive)
> [ParquetRecordReaderWrapper in 
> Hive|https://github.com/apache/hive/blob/e58b9d273cb78bda2947148bc54f4befb2514241/ql/src/java/org/apache/hadoop/hive/ql/io/parquet/read/ParquetRecordReaderWrapper.java#L221]
> It should use 
> [InternalParquetRecordReader|https://github.com/apache/incubator-parquet-mr/blob/master/parquet-hadoop/src/main/java/parquet/hadoop/InternalParquetRecordReader.java]
>  directly instead.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to