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

Atri Sharma commented on IGNITE-1006:
-------------------------------------

Hi,

Thanks for your comments.

Please see my reply below:

1) I was bit concerned about copying of nodes collection when called from 
hostNames() performance degradation but have now changed the implementation to 
use nodes() as you suggested.
2) Fixed tests.

Please see and let me know your thoughts and feedback.

> Add hostnames() method to ClusterGroup to get all hostnames in current 
> ClusterGroup
> -----------------------------------------------------------------------------------
>
>                 Key: IGNITE-1006
>                 URL: https://issues.apache.org/jira/browse/IGNITE-1006
>             Project: Ignite
>          Issue Type: Task
>            Reporter: Atri Sharma
>            Assignee: Atri Sharma
>             Fix For: sprint-7
>
>         Attachments: ignite-sprint-5_ignite-1006ver4.patch
>
>
> We should have method like nodes() in ClusterGroup to allow getting all 
> hostnames currently in this ClusterGroup. Method should ensure handling for 
> duplicates as well.



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

Reply via email to