tolbertam commented on code in PR #2007:
URL:
https://github.com/apache/cassandra-java-driver/pull/2007#discussion_r1936067748
##########
core/src/main/java/com/datastax/oss/driver/internal/core/metadata/InitialNodeListRefresh.java:
##########
@@ -63,22 +63,29 @@ public Result compute(
TokenFactory tokenFactory = null;
Map<UUID, DefaultNode> newNodes = new HashMap<>();
+ // Contact point nodes don't have host ID as well as other info yet, so we
fill them with node
+ // info found on first match by endpoint
+ Set<DefaultNode> matchedContactPoints = new HashSet<>();
Review Comment:
ah right interesting, didn't occur to me at the time, but made a similar
observation on the (initial
commit)[https://github.com/apache/cassandra-java-driver/pull/2007#discussion_r1934067152].
I suspect that for some reason this still works as I still see `Adding new
node {}` in my test, double checking this though.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
