I think it would have been clearer if we added a `unionCH` field and changed `getWriteConsistentHash()` to return that. It was already confusing that `getPendingCH()` meant one thing in Cluster/LocalTopologyManagerImpl and another thing in StateConsumer/ProviderImpl, and adding another field doesn't really help.