[ 
https://issues.apache.org/jira/browse/HBASE-17465?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15862125#comment-15862125
 ] 

Enis Soztutar commented on HBASE-17465:
---------------------------------------

bq. It is fine if you want to do this in a future patch.
Was looking at this again offline with [~xiaobingo]. We are using the base 
class only for master callables. So, let's leave this as it is in this patch. 

Together with this: {{if/Client.proto}}, I think we need to change the one in 
{{./hbase-protocol/src/main/protobuf/Client.proto}} (bin/copy-protobuf.sh 
copies this to under {{hbase-native-client/if/}} directory). 

- For the test, you do not need to re-use the XML creation from 
{{client-test.h}}. You can undo that, and instead create a client like this: 
{code}
Configuration conf;
hbase::Client client(conf);
{code}

Otherwise, the logic for exception handling, retries, timeouts, etc is pretty 
solid and very close to the existing impl. 


> [C++] implement request retry mechanism over RPC
> ------------------------------------------------
>
>                 Key: HBASE-17465
>                 URL: https://issues.apache.org/jira/browse/HBASE-17465
>             Project: HBase
>          Issue Type: Sub-task
>            Reporter: Xiaobing Zhou
>            Assignee: Xiaobing Zhou
>         Attachments: HBASE-17465-HBASE-14850.000.patch, 
> HBASE-17465-HBASE-14850.001.patch, HBASE-17465-HBASE-14850.002.patch, 
> HBASE-17465-HBASE-14850.003.patch, HBASE-17465-HBASE-14850.004.patch, 
> HBASE-17465-HBASE-14850.005.patch, HBASE-17465-HBASE-14850.006.patch, 
> HBASE-17465-HBASE-14850.007.patch
>
>
> HBASE-17051 implemented RPC layer. Requests retries will make system 
> reliable. This JIRA proposes adding it, which corresponds to similar 
> implementation in  SingleRequestCallerBuilder (or BatchCallerBuilder, 
> ScanSingleRegionCallerBuilder, SmallScanCallerBuilder, etc.) and 
> AsyncSingleRequestRpcRetryingCaller. As a bonus, retry should be more 
> generic, decoupled with HRegionLocation.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to