shotover / shotover-proxy

L7 data-layer proxy
https://docs.shotover.io
Apache License 2.0
84 stars 16 forks source link

Splitup cassandra connection abstraction #1680

Closed rukai closed 3 months ago

rukai commented 3 months ago

This PR only changes test code. This PR contains no functional changes to the tests.

cassandra/connection.rs is split up into connection.rs, scylla.rs, cdrs.rs and cpp.rs. This is done by introducing ScyllaConnection, CdrsConnection and CppConnection types that hold all wrapping logic for each driver.

I attempted to keep this as straightforward as possible, however there were some important renames that needed to be done, so I included those in this PR. The renames include:

Additionally the TIMEOUT constant is changed to include the whole Duration type. Previously it was not clear what unit of time the timeout was in.

codspeed-hq[bot] commented 3 months ago

CodSpeed Performance Report

Merging #1680 will not alter performance

Comparing rukai:splitup_cassandra_connection_abstraction (b1d6747) with main (11c0d58)

Summary

✅ 39 untouched benchmarks