fix: support existing deprecated Rebalance.Strategy field usage (#2352)

* for config backward compatibility remove the validation that prevented both fields being set and instead rely on precedence which favours the legacy field currently
IBM · Oct 3, 2022 · eaf1def · eaf1def
1 parent 619d4ef
commit eaf1def
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/config.go b/config.go
@@ -804,8 +804,6 @@ func (c *Config) Validate() error {
 		return ConfigurationError("Consumer.Group.Heartbeat.Interval must be < Consumer.Group.Session.Timeout")
 	case c.Consumer.Group.Rebalance.Strategy == nil && len(c.Consumer.Group.Rebalance.GroupStrategies) == 0:
 		return ConfigurationError("Consumer.Group.Rebalance.GroupStrategies or Consumer.Group.Rebalance.Strategy must not be empty")
-	case c.Consumer.Group.Rebalance.Strategy != nil && len(c.Consumer.Group.Rebalance.GroupStrategies) != 0:
-		return ConfigurationError("Consumer.Group.Rebalance.GroupStrategies and Consumer.Group.Rebalance.Strategy cannot be set at the same time")
 	case c.Consumer.Group.Rebalance.Timeout <= time.Millisecond:
 		return ConfigurationError("Consumer.Group.Rebalance.Timeout must be >= 1ms")
 	case c.Consumer.Group.Rebalance.Retry.Max < 0: