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

Mayuresh Gharat commented on KAFKA-1796:
----------------------------------------

[~nehanarkhede] Sure. Actually I went through the code and saw that the case 
for duplicate replicas for the same topic partition case has been handled (we 
were using the older one and came across this bug). But of course we can test 
this new reassignment tool once its ready. Thanks.

> Sanity check partition command line tools
> -----------------------------------------
>
>                 Key: KAFKA-1796
>                 URL: https://issues.apache.org/jira/browse/KAFKA-1796
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Guozhang Wang
>            Assignee: Mayuresh Gharat
>              Labels: newbie
>             Fix For: 0.8.3
>
>
> We need to sanity check the input json has the valid values before triggering 
> the admin process. For example, we have seen a scenario where the json input 
> for partition reassignment tools have partition replica info as {broker-1, 
> broker-1, broker-2} and it is still accepted in ZK and eventually lead to 
> under replicated count, etc. This is partially because we use a Map rather 
> than a Set reading the json input for this case; but in general we need to 
> make sure the input parameters like Json  needs to be valid before writing it 
> to ZK.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to