alternator: change default write isolation to only_rmw_uses_lwt

"always" is too restrictive and too slow to be a good default.
scylladb · Dec 1, 2020 · 9446bf9 · 9446bf9
1 parent cb144b2
commit 9446bf9
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/pkg/api/v1alpha1/cluster_webhook.go b/pkg/api/v1alpha1/cluster_webhook.go
@@ -39,10 +39,16 @@ func (r *ScyllaCluster) SetupWebhookWithManager(mgr ctrl.Manager) error {
 var _ webhook.Defaulter = &ScyllaCluster{}
 var _ webhook.Validator = &ScyllaCluster{}
 
+const (
+	AlternatorWriteIsolationAlways         = "always"
+	AlternatorWriteIsolationForbidRMW      = "forbid_rmw"
+	AlternatorWriteIsolationOnlyRMWUsesLWT = "only_rmw_uses_lwt"
+)
+
 func (c *ScyllaCluster) Default() {
 	if c.Spec.Alternator != nil {
 		if c.Spec.Alternator.WriteIsolation == "" {
-			c.Spec.Alternator.WriteIsolation = "always"
+			c.Spec.Alternator.WriteIsolation = AlternatorWriteIsolationOnlyRMWUsesLWT
 		}
 	}