absurdfarce commented on PR #1981: URL: https://github.com/apache/cassandra-java-driver/pull/1981#issuecomment-2596946379
Worth noting that this matches up with how [4.x does the same thing](https://github.com/apache/cassandra-java-driver/blob/4.x/core/src/main/java/com/datastax/oss/driver/internal/core/metadata/DefaultTopologyMonitor.java#L527-L536). There we check for "native_port" (and the DSE-specific "native_transport_port") when computing the broadcast RPC address, falling back to "rpc_port" if neither of them are found. For 4.x this was all re-worked when support for "native_transport_address" was added (see CASSANDRA-19352, which is actually now CASSJAVA-42) but if you look at the PR for that ticket you can see that "rpc_port" was used _before_ that change. -- 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]
