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 2AA319988 for ; Thu, 3 May 2012 21:47:15 +0000 (UTC) Received: (qmail 43544 invoked by uid 500); 3 May 2012 21:47:15 -0000 Delivered-To: apmail-hbase-issues-archive@hbase.apache.org Received: (qmail 43493 invoked by uid 500); 3 May 2012 21:47:15 -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 43484 invoked by uid 99); 3 May 2012 21:47:14 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 03 May 2012 21:47:14 +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; Thu, 03 May 2012 21:47:11 +0000 Received: from hel.zones.apache.org (hel.zones.apache.org [140.211.11.116]) by hel.zones.apache.org (Postfix) with ESMTP id 9622C42E926 for ; Thu, 3 May 2012 21:46:50 +0000 (UTC) Date: Thu, 3 May 2012 21:46:50 +0000 (UTC) From: "jiraposter@reviews.apache.org (JIRA)" To: issues@hbase.apache.org Message-ID: <822655166.24073.1336081610616.JavaMail.tomcat@hel.zones.apache.org> In-Reply-To: <1732844274.3183.1329893334374.JavaMail.tomcat@hel.zones.apache.org> Subject: [jira] [Commented] (HBASE-5444) Add PB-based calls to HMasterRegionInterface 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-5444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13267840#comment-13267840 ] jiraposter@reviews.apache.org commented on HBASE-5444: ------------------------------------------------------ bq. On 2012-05-02 23:43:07, Michael Stack wrote: bq. > src/main/java/org/apache/hadoop/hbase/master/HMaster.java, line 80 bq. > bq. > bq. > We need this? bq. bq. Gregory Chanan wrote: bq. I use it to convert the PB serverName that is passed into HMaster.regionServerReport into a ServerName that the ServerManager understands. Instead, we could have a ServerName static function that takes a PB ServerName and returns a ServerName. We already have a bunch of these parse* functions already, e.g. bq. bq. public static ServerName parseVersionedServerName(final byte [] versionedBytes) bq. bq. Michael Stack wrote: bq. Recently in trunk, we added a ServerName.parseFrom that should be able to make sense of any set of bytes parsed it whether pbs or old style versioned bytes. Could use this. I don't think this applies. From reading ServerName.parseFrom it looks like it requires the PBMagicPrefix, which this case doesn't have nor need. Am I missing something? - Gregory ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4463/#review7492 ----------------------------------------------------------- On 2012-05-02 23:19:20, Gregory Chanan wrote: bq. bq. ----------------------------------------------------------- bq. This is an automatically generated e-mail. To reply, visit: bq. https://reviews.apache.org/r/4463/ bq. ----------------------------------------------------------- bq. bq. (Updated 2012-05-02 23:19:20) bq. bq. bq. Review request for hbase and Michael Stack. bq. bq. bq. Summary bq. ------- bq. bq. Adds PB-based calls replacing HMasterRegionInterface. bq. bq. There are some temporary hacks, e.g. converting PB-based ServerLoad to existing HServerLoad so I didn't need to convert ClusterStatus (which brings in a lot of other changes). That will be cleaned up in HBASE-5445. bq. bq. bq. This addresses bug HBASE-5444. bq. https://issues.apache.org/jira/browse/HBASE-5444 bq. bq. bq. Diffs bq. ----- bq. bq. src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon 69434f7 bq. src/main/jamon/org/apache/hadoop/hbase/tmpl/regionserver/RSStatusTmpl.jamon 3c7c091 bq. src/main/java/org/apache/hadoop/hbase/ClusterStatus.java 5d7f07b bq. src/main/java/org/apache/hadoop/hbase/ServerLoad.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/ipc/HBaseRpcMetrics.java 0db2760 bq. src/main/java/org/apache/hadoop/hbase/ipc/HBaseServer.java 973c7cb bq. src/main/java/org/apache/hadoop/hbase/ipc/HMasterRegionInterface.java fd97830 bq. src/main/java/org/apache/hadoop/hbase/ipc/Invocation.java bb6ab3b bq. src/main/java/org/apache/hadoop/hbase/ipc/RegionServerStatusProtocol.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java f56127d bq. src/main/java/org/apache/hadoop/hbase/master/HMaster.java 81e9023 bq. src/main/java/org/apache/hadoop/hbase/master/MXBean.java 7f44dc2 bq. src/main/java/org/apache/hadoop/hbase/master/MXBeanImpl.java 45b8fe7 bq. src/main/java/org/apache/hadoop/hbase/master/MasterDumpServlet.java be63838 bq. src/main/java/org/apache/hadoop/hbase/master/ServerManager.java 80271b1 bq. src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java 994cb76 bq. src/main/java/org/apache/hadoop/hbase/protobuf/generated/HBaseProtos.java efcf74d bq. src/main/java/org/apache/hadoop/hbase/protobuf/generated/RegionServerStatusProtos.java PRE-CREATION bq. src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java ebffad6 bq. src/main/protobuf/RegionServerStatus.proto PRE-CREATION bq. src/main/protobuf/hbase.proto 12e6053 bq. src/main/resources/hbase-webapps/master/table.jsp 3ef1190 bq. src/test/java/org/apache/hadoop/hbase/MiniHBaseCluster.java 72554cb bq. src/test/java/org/apache/hadoop/hbase/coprocessor/TestClassLoading.java d039be3 bq. src/test/java/org/apache/hadoop/hbase/master/TestAssignmentManager.java 36046f8 bq. src/test/java/org/apache/hadoop/hbase/master/TestMXBean.java bd5fa90 bq. src/test/java/org/apache/hadoop/hbase/master/TestMasterNoCluster.java f8029ba bq. src/test/java/org/apache/hadoop/hbase/regionserver/TestServerCustomProtocol.java e99d251 bq. bq. Diff: https://reviews.apache.org/r/4463/diff bq. bq. bq. Testing bq. ------- bq. bq. Ran jenkins job, all unit tests passed. bq. bq. bq. Thanks, bq. bq. Gregory bq. bq. > Add PB-based calls to HMasterRegionInterface > -------------------------------------------- > > Key: HBASE-5444 > URL: https://issues.apache.org/jira/browse/HBASE-5444 > Project: HBase > Issue Type: Sub-task > Components: ipc, master, migration, regionserver > Reporter: Todd Lipcon > Assignee: Gregory Chanan > Fix For: 0.96.0 > > Attachments: HBASE-5444-v10-trunk.patch, HBASE-5444-v6-trunk.patch, HBASE-5444-v9-trunk.patch > > -- 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