ignite-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Artem Shutak (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (IGNITE-1006) Add hostnames() method to ClusterGroup to get all hostnames in current ClusterGroup
Date Mon, 22 Jun 2015 10:39:00 GMT

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

Artem Shutak commented on IGNITE-1006:
--------------------------------------

Hi Atri,

1. Why did you duplicate logic of ClusterGroupAdapter#nodes instead of reuse it for your goals?
2. We need to have tests for all changes, especially for new public methods. Please, add it.

> 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
>         Attachments: ignite-sprint-5_ignite-1006ver1.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)

Mime
View raw message