absurdfarce commented on code in PR #1915: URL: https://github.com/apache/cassandra-java-driver/pull/1915#discussion_r1587128032
########## core/src/main/java/com/datastax/oss/driver/internal/core/loadbalancing/LatencySensitiveLoadBalancingPolicy.java: ########## @@ -0,0 +1,275 @@ +/* Review Comment: I still need to give this a more complete review but at a minimum this class needs to go in the [examples directory](https://github.com/apache/cassandra-java-driver/tree/4.x/examples/src/main/java/com/datastax/oss/driver/examples) rather than inline with the default LBP like this. In 4.x the default LBP is intended to be the primary LBP used in nearly all situations; adding another LBP into the same package will significantly confuse that messaging. -- 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]
