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

Tsz Wo Nicholas Sze commented on RATIS-386:
-------------------------------------------

+1 the 004 patch looks good.

- 
./ratis-common/src/main/java/org/apache/ratis/protocol/RaftRetryFailureException.java:20:import
 org.apache.ratis.retry.RetryPolicy;:8: Unused import - 
org.apache.ratis.retry.RetryPolicy. [UnusedImports]
- 
./ratis-grpc/src/main/java/org/apache/ratis/grpc/GrpcConfigKeys.java:20:import 
org.apache.ratis.client.RaftClientConfigKeys;:8: Unused import - 
org.apache.ratis.client.RaftClientConfigKeys. [UnusedImports]

There are two unused imports.  Please remove them while committing.  Thanks, 
[~shashikant]!

> Raft Client Async API's should honor Retry Policy 
> --------------------------------------------------
>
>                 Key: RATIS-386
>                 URL: https://issues.apache.org/jira/browse/RATIS-386
>             Project: Ratis
>          Issue Type: Improvement
>          Components: client
>    Affects Versions: 0.3.0
>            Reporter: Shashikant Banerjee
>            Assignee: Shashikant Banerjee
>            Priority: Major
>             Fix For: 0.3.0
>
>         Attachments: RATIS-386.001.patch, RATIS-386.002.patch, 
> RATIS-386.003.patch, RATIS-386.004.patch
>
>
> Raft client sync Api has support for retry policies. Similarly, for Async 
> API's including watch Api, support for Retry Policy is required.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to