Lucas Kent

Fix RedisSinkCluster random node handling

1

1. RoutingInfo::Random does not route to a random node but to the first node in the list of master nodes 2. Many commands fallback to the default routing implementation of...

RedisSinkCluster self.masters sometimes contains replicas

2

I have observed a random test failure https://github.com/shotover/shotover-proxy/runs/6290958118?check_suite_focus=true occur because a RoutingInfo::Slot command was routed to a replica and the command "CLUSTER REPLICAS" must be run against a master. The...

Improve documentation infrastructure

2

Some ideas for avoiding broken and outdated documentation: * Assert that mdbook does not output any warnings - this should prevent internal broken links * Use a markdown parser to...

The error should contain the name of the command that is unsupported and we should return more specific errors when possible.

Redis pubsub support

2

In order to solve https://github.com/shotover/shotover-proxy/issues/537 we are going to have to design a way for shotover to handle the pubsub model which goes against many of the assumptions in shotovers...

Add instruction counter to benches

3

Unfortunately it seems unusable at the moment. I am getting instruction counts of ~7000 per message sent which sounds way too low. It must be failing to count instructions that...

Add cqlparser crate implemented by nom

2

So far only implements select and even then still doesnt have full coverage of select. However it does implement enough demonstrate the feasibility of writing a parser like this. ##...

The current benchmark system is alright. It can currently be used by developers to get a better understanding of how the system performs in general and the effects of their...

test_cassandra_peers_rewrite improvements

1

rewrite test_cassandra_peers_rewrite to do the following: * 3 node cassandra cluster with 3 corresponding shotover instances * topology yamls configured with CassandraPeersRewrite * current assertions we already have in test_cassandra_peers_rewrite...

In general transforms are supposed to handle any message type but there are some transforms that only make sense with one message type e.g. CassandraPeersRewrite and would indicate a user...