scylladb / cql-stress

8 stars 4 forks source link

Bump driver version to 0.12.0 #71

Closed muzarski closed 3 months ago

muzarski commented 3 months ago

Changes

Bumped driver's version to most recent 0.12.0.

Some code needed to be adjusted:

Consistency enum

Consistency enum supports two new variants: Serial and LocalSerial. Updated show_consistency_level accordingly.

Murmur3Partitioner

Murmur3Partitioner's API changed a bit, so its usage is adjusted in GeneratorConfig::new

new serialization api (0.11.0)

SerializedValues was renamed to LegacySerializedValues in serialization API refactor.

I changed the usage of SerializedValues simply to Vec<i64> in s-b's ReadOperation. I used Vec<i64> since both partition_key and clustering_keys are bigints.

Maybe it's better use LegacySerializedValues, but I'm not quite sure as I'm not familiar with the changes introduced in 0.11.0.

Lorak-mmk commented 3 months ago

new serialization api (0.11.0)

SerializedValues was renamed to LegacySerializedValues in serialization API refactor.

I changed the usage of SerializedValues simply to Vec<i64> in s-b's ReadOperation. I used Vec<i64> since both partition_key and clustering_keys are bigints.

Maybe it's better use LegacySerializedValues, but I'm not quite sure as I'm not familiar with the changes introduced in 0.11.0.

You should not use legacy structs, they will be removed in the future. Current approach (Vec<i64>) is ok of all your values are i64.