Hi Isuru, Thank you for the clarification . I will do the same.
Regards, Swapnil On Tue, Jun 2, 2015 at 2:14 PM, Isuru Haththotuwa <isu...@apache.org> wrote: > > > On Tue, Jun 2, 2015 at 12:57 PM, Swapnil Patil <swapnil.r...@gmail.com> > wrote: > >> Hi Isuru, >> >> Thanks for suggesstions. >> >> >>> 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. >>> >> >> Yes, I am aware of it. Currently I am using Eclipse. The thing is, I am >> new to Maven projects. I have imported all the projects in the workspace. >> But I am not able to find references to this class. I am getting the error >> "The resource is not on the build path of a Java project". To fix this I >> need to configure the build path. But unfortunately, the option to >> configure build path is disabled. Now I am not sure why is this so. This >> option to configure build path is enabled for a few projects (which I >> assume Eclipse treat them as Java project) but for others (many of them) it >> is disabled (I assume Eclipse doesn't treat them as Java projects). >> >> I am not sure about this behaviour. Is this because I have imported Maven >> projects? Is there anything else I need to do? like creating Eclipse >> projects manually with command 'maven eclipse:eclipse'? >> > Yes, mvn eclipse:eclipse should create the eclipse project for you. Then, > open the project from the Eclipse IDE. It should work. > >> >> >> >>> 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. >>> >> >> Okay. Thanks. I think this new field should be optional. >> >> Thanks and Regards, >> Swapnil >> >> 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 >>>> >>>> -- >>>> <%2B94%20716%20358%20048> >>>> <%2B94%20716%20358%20048> >>>> Thanks and Regards, >>>> >>>> Isuru H. >>>> <%2B94%20716%20358%20048> >>>> +94 716 358 048* <http://wso2.com/>* >>>> >>>> >>>> * <http://wso2.com/>* >>>> >>>> >>>>