[jira] Commented: (ZOOKEEPER-248) QuorumPeer should use Map interface instead of HashMap implementation.

2008-12-10 Thread Hudson (JIRA)

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

Hudson commented on ZOOKEEPER-248:
--

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


> QuorumPeer should use Map interface instead of HashMap implementation.
> --
>
> Key: ZOOKEEPER-248
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-248
> Project: Zookeeper
>  Issue Type: Improvement
>  Components: quorum
>Affects Versions: 3.0.0, 3.0.1
>Reporter: Nitay Joffe
>Assignee: Nitay Joffe
>Priority: Trivial
> Fix For: 3.1.0
>
> Attachments: 248.patch
>
>
> Using HashMap directly to store quorumPeers member variable unnecessarily 
> decides for the user what implementation of Map to use.

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



[jira] Commented: (ZOOKEEPER-248) QuorumPeer should use Map interface instead of HashMap implementation.

2008-12-08 Thread Mahadev konar (JIRA)

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

Mahadev konar commented on ZOOKEEPER-248:
-

+1 the patch looks good... 

> QuorumPeer should use Map interface instead of HashMap implementation.
> --
>
> Key: ZOOKEEPER-248
> URL: https://issues.apache.org/jira/browse/ZOOKEEPER-248
> Project: Zookeeper
>  Issue Type: Improvement
>  Components: quorum
>Affects Versions: 3.0.0, 3.0.1
>Reporter: Nitay Joffe
>Assignee: Nitay Joffe
>Priority: Trivial
> Fix For: 3.1.0
>
> Attachments: 248.patch
>
>
> Using HashMap directly to store quorumPeers member variable unnecessarily 
> decides for the user what implementation of Map to use.

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