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