Major Description
Findbugs spotted this issue where two strings are compared by identity instead of content:
/** * Return the first occurrence for the given role, or null if there is none. */ private Replica getOneOfRoleOrNull(Role role) { for (Replica r : replicas) { if (r.getRole() == role.toString()) { return r; } } return null; }
it's not clear why strings are being used for comparison at all rather than checking enum value equality, which would be both faster and more likely to be correct. It may be that this code works fine despite the sketchiness, though, because the two string objects are likely to be derived from the same enumValue.toString() which returns a constant string object.