Return-Path: X-Original-To: apmail-hbase-issues-archive@www.apache.org Delivered-To: apmail-hbase-issues-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 9DF4D95CB for ; Tue, 14 Feb 2012 21:54:24 +0000 (UTC) Received: (qmail 77759 invoked by uid 500); 14 Feb 2012 21:54:24 -0000 Delivered-To: apmail-hbase-issues-archive@hbase.apache.org Received: (qmail 77691 invoked by uid 500); 14 Feb 2012 21:54:24 -0000 Mailing-List: contact issues-help@hbase.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Delivered-To: mailing list issues@hbase.apache.org Received: (qmail 77367 invoked by uid 99); 14 Feb 2012 21:54:23 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 14 Feb 2012 21:54:23 +0000 X-ASF-Spam-Status: No, hits=-2000.0 required=5.0 tests=ALL_TRUSTED,T_RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.116] (HELO hel.zones.apache.org) (140.211.11.116) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 14 Feb 2012 21:54:22 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id CE0011B79A3 for ; Tue, 14 Feb 2012 21:54:02 +0000 (UTC) Date: Tue, 14 Feb 2012 21:54:02 +0000 (UTC) From: "jiraposter@reviews.apache.org (Commented) (JIRA)" To: issues@hbase.apache.org Message-ID: <1242163806.38125.1329256442845.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <1171051132.33879.1329172383425.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (HBASE-5394) Add ability to include Protobufs in HbaseObjectWritable MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/HBASE-5394?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13208052#comment-13208052 ] jiraposter@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 21:53:00.016924) Review request for hbase and Todd Lipcon. Changes ------- Addressed Todd's comment. 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 > > 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