What's the advantage of creating a HostAndPort and using its toString, rather than appending the strings ourself? I guess it's giving us null-checks for hostname and thriftPort. If it is null, then good we detect and report it - but do we really want to fail setting the cassandra_cluster_nodes entirely? Not sure it's worth much extra coding though, as if service_up then we shouldn't be getting nulls...