[jira] Commented: (ZOOKEEPER-407) address all findbugs warnings in org.apache.zookeeper.server.quorum.** packages

2009-05-21 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12711892#action_12711892
 ] 

Hudson commented on ZOOKEEPER-407:
--

Integrated in ZooKeeper-trunk #321 (See 
[http://hudson.zones.apache.org/hudson/job/ZooKeeper-trunk/321/])


> 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, 
> 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.



[jira] Commented: (ZOOKEEPER-407) address all findbugs warnings in org.apache.zookeeper.server.quorum.** packages

2009-05-19 Thread Flavio Paiva Junqueira (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12710794#action_12710794
 ] 

Flavio Paiva Junqueira commented on ZOOKEEPER-407:
--

For this patch I'm certainly not proposing that we fix any problem with exit 
calls. All modifications I have suggested are to get rid of the findbugs 
warnings. In fact, my preference is to leave the exit calls as they are and 
perhaps deal with them in a separate jira if there is any need to do so.  


 

> 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.



[jira] Commented: (ZOOKEEPER-407) address all findbugs warnings in org.apache.zookeeper.server.quorum.** packages

2009-05-19 Thread Patrick Hunt (JIRA)

[ 
https://issues.apache.org/jira/browse/ZOOKEEPER-407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12710787#action_12710787
 ] 

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 
particular)
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.