Re: Deprecate IgniteConfiguration.getGridName

2016-03-02 Thread bintisepaha
Thank you Val, this is very helpful. -- View this message in context: http://apache-ignite-developers.2346864.n4.nabble.com/Deprecate-IgniteConfiguration-getGridName-tp2420p7685.html Sent from the Apache Ignite Developers mailing list archive at Nabble.com.

Re: Deprecate IgniteConfiguration.getGridName

2016-02-26 Thread Valentin Kulichenko
Binti, This is not an issue. What is discussed here is just proper naming of this configuration property, but the concept stays the same - this is the name of local Ignite instance and there is no requirement for all nodes in topology to share the same name. Whether nodes discover each other or n

Re: Deprecate IgniteConfiguration.getGridName

2016-02-26 Thread bintisepaha
Hi, We are facing this issue, nodes with different grid names are being identified in a single topology. Did the fix for this make it into version 1.6? If yes, how can I get this version? Thanks, Binti -- View this message in context: http://apache-ignite-developers.2346864.n4.nabble.com/Depr

Re: Deprecate IgniteConfiguration.getGridName

2015-08-26 Thread Valentin Kulichenko
+1 for 'instanceName'. 'nodeName' looks confusing as well - I would think that it should be unique among all nodes. -Val On Wed, Aug 26, 2015 at 2:25 AM, Raul Kripalani wrote: > Agree. The term grid to refer to a single node is very confusing. > On 19 Aug 2015 08:58, "Semyon Boikov" wrote: > >

Re: Deprecate IgniteConfiguration.getGridName

2015-08-26 Thread Raul Kripalani
Agree. The term grid to refer to a single node is very confusing. On 19 Aug 2015 08:58, "Semyon Boikov" wrote: > In my opinion this propery just should be renamed to 'nodeName'. > Possibility to set node name is very useful in our tests. As I see > 'setNodeId' is already deprecated in IgniteConfi

Re: Deprecate IgniteConfiguration.getGridName

2015-08-26 Thread Sergi Vladykin
I think `instanceName` is good enough (if it will have a proper javadoc). Sergi 2015-08-26 9:24 GMT+03:00 Semyon Boikov : > We can add 'nodeName' on ClusterNode API, it is very easy to implement > since node name is already stored in node attributes. > > On Wed, Aug 26, 2015 at 4:44 AM, Dmitriy

Re: Deprecate IgniteConfiguration.getGridName

2015-08-25 Thread Semyon Boikov
We can add 'nodeName' on ClusterNode API, it is very easy to implement since node name is already stored in node attributes. On Wed, Aug 26, 2015 at 4:44 AM, Dmitriy Setrakyan wrote: > Although I share the sentiment that gridName is confusing, I think renaming > gridName to nodeName will be equa

Re: Deprecate IgniteConfiguration.getGridName

2015-08-25 Thread Dmitriy Setrakyan
Although I share the sentiment that gridName is confusing, I think renaming gridName to nodeName will be equally confusing. The "nodeName" property sounds like something that should be on ClusterNode API, which is not the case. If we rename if in one of the future versions, I think instanceName wo