search

scylladb / scylla-go-driver

Experimental, high performance Scylla Driver, University of Warsaw students' project
Apache License 2.0
151 stars 7 forks source link

session: prepare now prepares on all nodes #266

Closed Kulezi closed 2 years ago

Kulezi commented 2 years ago

Prepare now prepares on all nodes, once per node, as sources suggest: https://github.com/apache/cassandra/blob/adcff3f630c0d07d1ba33bf23fcb11a6db1b9af1/doc/native_protocol_v4.spec#L738-L740 https://github.com/scylladb/scylladb/blob/4d24097b4b86d484159c011f46b9e43268d342d1/transport/server.cc#L941-L950

Also made the driver fallback from token aware policy to round robin if both session keyspace and query keyspace are unspecified, this is a temporary fix until #257 is resolved.

Kulezi commented 2 years ago

Applied changes suggested by @piodul, also removed leaking goroutines from TLS integration test caused by unclosed session.