rvansa commented on PR #2034: URL: https://github.com/apache/cassandra-java-driver/pull/2034#issuecomment-2837876708
@absurdfarce Shouldn't these statistics that affect load balancing be reset when you force the nodes down? If that bit is missing, could you point me to parts of code that I should adjust, or ideally a test that validates a behaviour that uses those? Normally I would try to keep PR minimal, but if this would be severe problem I can try to address that. I think that the discussion of whether it is worth is a bit vague when we don't have data to back up the claims. But we're touching something @aratno mentioned above - CRaC needs all parts of the application to be ready. The application might not be based around the Cassandra driver, that could be just a small part of its interface to a larger system. The driver setup might not be performance bottleneck at all, so maybe we're not saving so much on this front, but enable savings in a completely different part of the application. That's why my focus in here would be correctness, not performance. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org
