Rename Assembly.size() method to avoid confusion with extensions.

coherence-community / oracle-bedrock

Oracle Bedrock

Other

55 stars 31 forks source link

Rename Assembly.size() method to avoid confusion with extensions. #246

Closed brianoliver closed 8 years ago

brianoliver commented 9 years ago

The current Assembly.size() method can be confusing to developers calling extensions of the AbstractAssembly or implementing the Assembly interface (eg: Coherence-based Cluster implementations) as it return the number Applications in an Assembly, not the Cluster size.

To avoid this confusion we need to rename the size() method (or remove it). Calling it getAssemblySize() or getApplicationCount() would be much better as it would remove any possible confusion.

brianoliver commented 8 years ago

@brianoliver said: Removed the method as;

a). it wasn't used anywhere b). it was confusing

brianoliver commented 8 years ago

This issue was imported from JIRA ORACLETOOLS-246

brianoliver commented 9 years ago

Reported by @brianoliver

brianoliver commented 8 years ago

Marked as fixed by @brianoliver on Wednesday, April 6th 2016, 3:39:12 pm