tolbertam commented on code in PR #4699: URL: https://github.com/apache/cassandra/pull/4699#discussion_r3069691242
########## conf/cassandra.yaml: ########## @@ -2682,6 +2682,20 @@ max_security_label_length: 48 # compressed by dictionary compressor and training_min_frequency is set to 0m (the default when unset). #unset_training_min_frequency_enabled: true +# Minimum client driver versions. Connections from drivers whose version is below +# the configured minimum will be warned or rejected. Connections that do not report +# a driver name or version are considered valid. The map key is the driver name Review Comment: > It is not like you are running drivers without any driver id every other day that this would be so common it would need to have a lot of attention This does happen occasionally for us, we help partner teams by supporting specific drivers, but we don't currently mandate what gets used. * Clients not sending driver versions: We've had some occasions where a team is using a very old driver that doesn't send driver name/version and it would be nice to have some enforcement/visibility around that. * Clients using client libraries we don't know about: Using a language like rust that has a community maintained driver that does send this information (https://github.com/scylladb/scylla-rust-driver). In either case having some visibility via the warning guardrail could be useful. -- 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]
