Patrick Hunt commented on ZOOKEEPER-407:

Take a look at what I did in ZOOKEEPER-410, I basically allowed a couple of 
exceptions where I thought:
1) big impact if we didn't allow (potential instability/incorrectness in 
2) I thought it was ok to exit if we; clearly documented and esp logged as 
FATAL the cause
3) command line driver code (ie main()) is ok to use exit

it's not clear to me that in server code (vs library) exit should be entirely 
disallowed. I'm a follower of the
"fail fast" approach - I'd rather see us exit with useful feedback than linger. 
In library code obv you don't
want to exit as this may not be what all clients want to do - but for servers I 
think it's ok, esp if we review/approve
each exception (which we already do).

> address all findbugs warnings in org.apache.zookeeper.server.quorum.** 
> packages
> -------------------------------------------------------------------------------
>                 Key: ZOOKEEPER-407
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-407
>             Project: Zookeeper
>          Issue Type: Sub-task
>          Components: server
>            Reporter: Patrick Hunt
>            Assignee: Flavio Paiva Junqueira
>             Fix For: 3.2.0
>         Attachments: ZOOKEEPER-407.patch, ZOOKEEPER-407.patch
> Flavio please address the findbugs warnings in quorum and sub packages. See
> http://hudson.zones.apache.org/hudson/view/ZooKeeper/job/ZooKeeper-trunk/308/violations/
> for a list of issues or use ant's findbugs (or eclipse) to identify the 
> issues.
> In general we should try to fix these, but ignoring the warning (annotation) 
> is ok if it includes comments to effect why, also has
>  to be reviewed/approved.

This message is automatically generated by JIRA.
You can reply to this email to add a comment to the issue online.

Reply via email to