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

ASF GitHub Bot commented on KAFKA-4053:
---------------------------------------

Github user asfgit closed the pull request at:

    https://github.com/apache/kafka/pull/1751


> Refactor TopicCommand to remove redundant if/else statements
> ------------------------------------------------------------
>
>                 Key: KAFKA-4053
>                 URL: https://issues.apache.org/jira/browse/KAFKA-4053
>             Project: Kafka
>          Issue Type: Improvement
>          Components: admin
>    Affects Versions: 0.10.0.1
>            Reporter: Shuai Zhang
>            Priority: Minor
>             Fix For: 0.10.1.0
>
>
> In TopicCommand, there are a lot of redundant if/else statements, such as
> ```val ifNotExists = if (opts.options.has(opts.ifNotExistsOpt)) true else 
> false```
> We can refactor it as the following statement:
> ```val ifNotExists = opts.options.has(opts.ifNotExistsOpt)```



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

Reply via email to