search

thiagotts / CloudReports

An extensible simulation tool for energy-aware cloud computing environments
GNU General Public License v3.0
44 stars 51 forks source link

RoundRobinDatacenterBroker- getDatacenterId() seems this method is not used #10

Closed laxmisuresh closed 9 years ago

laxmisuresh commented 9 years ago

I have put the break point in the RoundRobinDatacenterBroker class, getDatacenterId method. The control is not coming to this method. I believe that this method is not used anywhere in the cloud reports. Kindly clarify. The control is coming to constructor and getDatacenterIdList() method but not to getDatacenterId method. Kindly clarify the use of this method.

@Override public int getDatacenterId() { currentId++; int index = currentId%getDatacenterIdsList().size(); return getDatacenterIdsList().get(index); }

Regards, Suresh.S

thiagotts commented 9 years ago

You are right. The getDatacenterId method is not used anymore inside the project and was left there due to compatibility issues with previous versions of CloudSim. It was previously used by the class org.cloudbus.cloudsim.DatacenterBroker (from CloudSim framework) to get new ids for datacenters to be managed by that broker. The latest versions of DatacenterBroker does not use this method anymore, so it can be removed from CloudReports in future versions.