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 3D73C1007C for ; Sat, 6 Dec 2014 18:48:13 +0000 (UTC) Received: (qmail 96582 invoked by uid 500); 6 Dec 2014 18:48:13 -0000 Delivered-To: apmail-hbase-issues-archive@hbase.apache.org Received: (qmail 96527 invoked by uid 500); 6 Dec 2014 18:48:13 -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 96514 invoked by uid 99); 6 Dec 2014 18:48:13 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 06 Dec 2014 18:48:13 +0000 Date: Sat, 6 Dec 2014 18:48:12 +0000 (UTC) From: "Jurriaan Mous (JIRA)" To: issues@hbase.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (HBASE-12597) Add RpcClient interface and enable changing of RpcClient implementation 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-12597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14236921#comment-14236921 ] Jurriaan Mous commented on HBASE-12597: --------------------------------------- Here you go. :) I hope that the cluster tests all work out. And thanks for all your input! Was nice to learn how it all works here. Am looking forward to run the included RpcClient tests on my new client after I adopt the new RpcClient interface. Nice way to test all kinds of failure situations. > Add RpcClient interface and enable changing of RpcClient implementation > ----------------------------------------------------------------------- > > Key: HBASE-12597 > URL: https://issues.apache.org/jira/browse/HBASE-12597 > Project: HBase > Issue Type: Improvement > Components: Client > Reporter: Jurriaan Mous > Fix For: 1.0.0, 2.0.0 > > Attachments: HBASE-12597-v1.patch, HBASE-12597-v2.patch, HBASE-12597-v3.patch, HBASE-12597-v4.patch, HBASE-12597-v4.patch, HBASE-12597-v5.patch, HBASE-12597-v6.patch, HBASE-12597-v7.patch, HBASE-12597-v8.patch, HBASE-12597.patch > > > Currently HConnectionImplementation works with the included RpcClient which is a direct implementation and not defined by an interface. > It would be great to be able to swap out the default RpcClient with another implementation which can also be controlled by the default HConnectionImplementation. > Suggested changes: > - Create a RpcClient interface which defines all the ways HConnectionImplementation interacts with an RPC client. Like getting a blocking protobuf service interface or closing the client. > - Define which RpcClient implementation to construct by setting a configuration variable which defaults to the current RpcClient. > - Possibly create an abstract RpcClient class to only load all the basic Rpc layer configurations to be used in an implementation. > Why? It enables experimentation with RpcClients which could enable new features or could be more performant than the included client. > I created a new RpcClient implementation based on Netty which can also be called asynchronously. It would be great to also be able to use this RpcClient in all the default ways and tests to see if there are any issues with it. > https://github.com/jurmous/async-hbase-client/ > https://github.com/jurmous/async-hbase-client/blob/master/src/main/java/org/apache/hadoop/hbase/ipc/AsyncRpcClient.java -- This message was sent by Atlassian JIRA (v6.3.4#6332)