goCQL driver accepts wrong case datacenter which disables lwt partition routing optimization

scylladb / scylla-go-driver

Experimental, high performance Scylla Driver, University of Warsaw students' project

Apache License 2.0

151 stars 7 forks source link

goCQL driver accepts wrong case datacenter which disables lwt partition routing optimization #302

Closed pdbossman closed 3 months ago

pdbossman commented 3 months ago

When data center in schema is lower case and goCQL is passed an uppercase dc, lwt routing optimization is lost. Eg. cluster.PoolConfig.HostSelectionPolicy = gocql.TokenAwareHostPolicy(gocql.DCAwareRoundRobinPolicy("DC1"))

I recommend you set it as this: cluster.PoolConfig.HostSelectionPolicy = gocql.TokenAwareHostPolicy(gocql.DCAwareRoundRobinPolicy("dc1"))

goCQL should error with incorrect data center name.

cc: @hopugop can you attach the reproducer please.

pdbossman commented 3 months ago

Sorry, wrong repo