-
Bug
-
Resolution: Done
-
Critical
-
4.2.1.FINAL
-
None
I'm not exactly sure how to reproduce it and why it happened but at least the problem is a quality / error handling issue.
TopologyAwareConsistentHash can run in an endless loop when rehashing runs and 'numOwners' is greater than the available addresses in 'processSequence'.
The endless loop is in the method getOwners() and the reason for it is shown in the simplified snippet below. It would really be helpful if the method either throws an exception with an exact definition what went wrong (e.g. including the address list and numOwners count) or handle the case if it's normal that this can happen when the cluster is just about to be built up.
private List<Address> getOwners(Address address, int numOwners) { ... ArrayList<Address> processSequence = ... ... List<Address> result = new ArrayList<Address>(); while (result.size() < numOwners) { Iterator<Address> addrIt = processSequence.iterator(); while (addrIt.hasNext()) { result.add(addrIt.next()); addrIt.remove(); } } return result; }