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

Mukul Kumar Singh commented on RATIS-783:
-----------------------------------------

+1, the patch looks good to me.

> RaftServerProxy#groupAddAsync removes the group on AlreadyExistsException
> -------------------------------------------------------------------------
>
>                 Key: RATIS-783
>                 URL: https://issues.apache.org/jira/browse/RATIS-783
>             Project: Ratis
>          Issue Type: Bug
>          Components: server
>            Reporter: Nanda kumar
>            Assignee: Nanda kumar
>            Priority: Major
>         Attachments: RATIS-783.000.patch
>
>
> {{RaftServerProxy#groupAddAsync}} removes the existing group on 
> {{AlreadyExistsException}}.
> We can log WARN message, but we should not remove the existing group.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to