-1, this looks ugly when we have a CacheTopology with 2 ConsistentHashes inside. Better not to log the segment owners at all in toString(), and call another method to log them explicitly when the ConsistentHash is created in ClusterTopologyManagerImpl or in LocalTopologyManagerImpl (and make those log messages TRACE-only).