shotover / shotover-proxy

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

Add cassandra java driver to test suite #1681

Closed rukai closed 2 months ago

rukai commented 3 months ago

This PR implements support for the cassandra java driver for a subset of our cassandra integration tests:

Other tests do not run with the cassandra driver due to missing features in the cassandra driver abstraction. We can come back to these at a later time.

NATIVE_COL_TYPES becomes fn supported_native_col_types so that we can disable the Decimal and Varint types for the cassandra driver. We will need to do some complex refactoring to enable those types in the cassandra driver, so its left out of scope of this PR.

Again, since we cant support decimal from the java driver right now we avoid testing decimals in prepared_statement_all.rs like we already do for the cpp driver.

Various functionality of the java driver is not exposed (e.g. TLS, protocol selection) since they arent needed for the initial tests that we are supporting. The java driver wrapper intentionally panics when these functionalities are enabled to ensure that it does not silently misbehave.

codspeed-hq[bot] commented 3 months ago

CodSpeed Performance Report

Merging #1681 will not alter performance

Comparing rukai:add_cassandra_java_driver (d03d64c) with main (374ef8c)

Summary

✅ 39 untouched benchmarks