Hi Swapnil,

Member class is used in the Topology hierarchy, to represent each member of
a cluster. For me, the easiest way to find where it has been referenced is
using the IDE that you are using. For an example, in IntelliJ Idea, you can
right click on the Member class's constructor -> click on find usages. I'm
sure you know this already.

IMO If this instance id is an optional field, its fine to use a setter. If
its a mandatory field for all Member objects, the it should be initialized
in the same constructor that is there currently.

On Mon, Jun 1, 2015 at 8:33 PM, Swapnil Patil <swapnil.r...@gmail.com>
wrote:

> Hi Devs,
>
> I am adding a new field
> in org.apache.stratos.messaging.domain.topology.Member class, called
> instanceId.
>
> For this, what would be a better approach
>
> 1. Modifying the existing constructor
> 2. Adding a new constructor which takes this as an additional argument
> 3. Just have a setter method.
>
> In case of 2 and 3, the member may not be initialized properly.
>
> I am confused about which to use because I am unaware of the scope of this
> Member class. If this is widely used then option 1 would mean changes at a
> lot of places.
>
> If I know what is the scope of this Member class, it would help me
> determine which approach to follow. Is there any way to find out in what
> all places a class's object is used apart from searching it manually
> through the codebase?
>
> I would appreciate any help in deciding the approach.
>
> Thanks and Regards,
> Swapnil
>
> --
> Thanks and Regards,
>
> Isuru H.
> +94 716 358 048* <http://wso2.com/>*
>
>
> * <http://wso2.com/>*
>
>
>

Reply via email to