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

Xuefu Zhang commented on HIVE-7022:
-----------------------------------

None of the test failures seem related. Patch is ready to be reviewed. 
[~brocknoland] Do you mind taking a look when you get a chance?

> Replace BinaryWritable with BytesWritable in Parquet serde
> ----------------------------------------------------------
>
>                 Key: HIVE-7022
>                 URL: https://issues.apache.org/jira/browse/HIVE-7022
>             Project: Hive
>          Issue Type: Improvement
>          Components: Serializers/Deserializers
>    Affects Versions: 0.13.0
>            Reporter: Xuefu Zhang
>            Assignee: Xuefu Zhang
>         Attachments: HIVE-7022.patch
>
>
> Currently ParquetHiveSerde uses BinaryWritable to enclose bytes read from 
> Parquet data. However, existing Hadoop class, BytesWritable, already does 
> that, and BinaryWritable offers no advantage. On the other hand, 
> BinaryWritable has a confusing getString() method, which, in misused, can 
> cause unexpected result. The proposal here is to replace it with Hadoop 
> BytesWritable.
> The issue was identified in HIVE-6367, serving as a follow-up JIRA. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to