[ https://issues.apache.org/jira/browse/KAFKA-1796?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14233913#comment-14233913 ]
Neha Narkhede edited comment on KAFKA-1796 at 12/4/14 5:01 AM: --------------------------------------------------------------- [~guozhang], KAFKA-1678 is the umbrella JIRA for all the improvements. It will be great if you can take a quick look and review it. See if you'd like to add anything to it. was (Author: nehanarkhede): [~guozhang], Here is the umbrella JIRA for all the improvements. It will be great if you can take a quick look and review it. See if you'd like to add anything to it. > 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 > 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)