You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@zookeeper.apache.org by "Jinjiang Ling (Jira)" <ji...@apache.org> on 2020/03/17 00:53:00 UTC

[jira] [Updated] (ZOOKEEPER-3760) remove a useless throwing CliException

     [ https://issues.apache.org/jira/browse/ZOOKEEPER-3760?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jinjiang Ling updated ZOOKEEPER-3760:
-------------------------------------
    Attachment: ZOOKEEPER-3760-1.patch

> remove a useless throwing CliException
> --------------------------------------
>
>                 Key: ZOOKEEPER-3760
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-3760
>             Project: ZooKeeper
>          Issue Type: Bug
>    Affects Versions: 3.5.7
>            Reporter: Jinjiang Ling
>            Priority: Major
>         Attachments: ZOOKEEPER-3760-1.patch
>
>
> when I upgrade zookeeper from 3.4.13 to 3.5.7 in my application, I find the function processCmd in ZooKeeperMain.java is just like blow
> {code:java}
> protected boolean processCmd(MyCommandOptions co) throws CliException, IOException, InterruptedException {
>     boolean watch = false;
>     try {
>         watch = processZKCmd(co);
>         exitCode = ExitCode.EXECUTION_FINISHED.getValue();
>     } catch (CliException ex) {
>         exitCode = ex.getExitCode();
>         System.err.println(ex.getMessage());
>     }
>     return watch;
> }
> {code}
> it throws {color:#FF0000}CliException {color}which has been caught in the funciton, so I think it can be removed.



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