scylladb / scylla-cqlsh

A fork of the cqlsh code
Apache License 2.0
16 stars 32 forks source link

cqlsh should use cql v4 by default when connecting #44

Closed mykaul closed 1 year ago

mykaul commented 1 year ago

This is what ScyllaDB supports, and if it just inherits the default from the Python driver, it tries with different versions (66, 65, 5) until it finally succeeds, which just slows down the connection. No reason for ScyllaDB's cqlsh not to be opinionated about the version. I think the line:

optvalues.protocol_version = option_with_default(configs.getint, 'protocol', 'version', None)

Should be changed, for the protocol version to be something other than None (probably 4).