exec_profile: Default to SerialConsistency::LocalSerial

wprzytula · wprzytula · commit 46f0fcdd9dd8 · 2023-05-25T14:39:36.000+02:00
Contrary to what is stated in documentation, the default
SerialConsistency was None, which resulted in SerialConsistency not
being set in the CQL frames. This led to ProtocolError in case of LWT
statements. As it is better to always include SerialConsistency (even
for statements for which it is not relevant), the previous default is
reverted.
diff --git a/scylla/src/transport/execution_profile.rs b/scylla/src/transport/execution_profile.rs
@@ -183,7 +183,7 @@ pub(crate) mod defaults {
         Consistency::LocalQuorum
     }
     pub fn serial_consistency() -> Option<SerialConsistency> {
-        None
+        Some(SerialConsistency::LocalSerial)
     }
     pub fn request_timeout() -> Option<Duration> {
         Some(Duration::from_secs(30))

Original file line number	Diff line number	Diff line change
`@@ -183,7 +183,7 @@ pub(crate) mod defaults {`
`183`	`183`	`Consistency::LocalQuorum`
`184`	`184`	`}`
`185`	`185`	`pub fn serial_consistency() -> Option<SerialConsistency> {`
`186`		`- None`
	`186`	`+ Some(SerialConsistency::LocalSerial)`
`187`	`187`	`}`
`188`	`188`	`pub fn request_timeout() -> Option<Duration> {`
`189`	`189`	`Some(Duration::from_secs(30))`