[ https://issues.apache.org/jira/browse/HBASE-5394?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13208114#comment-13208114 ]
jirapos...@reviews.apache.org commented on HBASE-5394: ------------------------------------------------------ ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3898/ ----------------------------------------------------------- (Updated 2012-02-14 23:32:59.226886) Review request for hbase and Todd Lipcon. Changes ------- Addressed Stack's comments Summary ------- Port HADOOP-7379 to HBase so that HBase WritableRpcEngine can support PB-based types, in addition to Writables. This is one of the first steps for wire compatibility. This addresses bug HBASE-5394. https://issues.apache.org/jira/browse/HBASE-5394 Diffs (updated) ----- src/main/java/org/apache/hadoop/hbase/io/DataOutputOutputStream.java PRE-CREATION src/main/java/org/apache/hadoop/hbase/io/HbaseObjectWritable.java 3c71192 src/main/java/org/apache/hadoop/hbase/util/ProtoUtil.java PRE-CREATION src/test/java/org/apache/hadoop/hbase/ipc/TestPBOnWritableRpc.java PRE-CREATION Diff: https://reviews.apache.org/r/3898/diff Testing ------- The TestPBOnWritableRpc unit test tests using PB-based type and Writable with WritableRpcEngine. Thanks, Jimmy > Add ability to include Protobufs in HbaseObjectWritable > ------------------------------------------------------- > > Key: HBASE-5394 > URL: https://issues.apache.org/jira/browse/HBASE-5394 > Project: HBase > Issue Type: Improvement > Reporter: Zhihong Yu > Assignee: Jimmy Xiang > Attachments: hbase-5394.txt > > > This is a port of HADOOP-7379 > This is to add the cases to HbaseObjectWritable to handle subclasses of > Message, the superclass of codegenned protobufs. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira