This is going to copy the serverHolders into a List on every call, effectively making this an O(n) operation with O(n) extra memory.  I don't believe we need that.