Skip to content

Commit

Permalink
fix missing returned errors
Browse files Browse the repository at this point in the history
Signed-off-by: Thomas Labarussias <[email protected]>
  • Loading branch information
Issif authored and poiana committed Jul 26, 2023
1 parent 4d11678 commit 5cd6c34
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion outputs/kafka.go
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,7 @@ func NewKafkaClient(config *types.Configuration, stats *types.Statistics, promSt
case "round_robin":
kafkaWriter.Balancer = &kafka.RoundRobin{}
default:
log.Printf("[ERROR] : Kafka - unsupported balancer %q\n", config.Kafka.Balancer)
return nil, fmt.Errorf("unsupported balancer %q", config.Kafka.Balancer)
}

Expand All @@ -97,6 +98,7 @@ func NewKafkaClient(config *types.Configuration, stats *types.Statistics, promSt
case "NONE":
// leave as default, none
default:
log.Printf("[ERROR] : Kafka - unsupported compression %q\n", config.Kafka.Compression)
return nil, fmt.Errorf("unsupported compression %q", config.Kafka.Compression)
}

Expand All @@ -108,7 +110,8 @@ func NewKafkaClient(config *types.Configuration, stats *types.Statistics, promSt
case "NONE":
kafkaWriter.RequiredAcks = kafka.RequireNone
default:
return nil, fmt.Errorf("unsupported required ACKs %q", config.Kafka.Compression)
log.Printf("[ERROR] : Kafka - unsupported required ACKs %q\n", config.Kafka.RequiredACKs)
return nil, fmt.Errorf("unsupported required ACKs %q", config.Kafka.RequiredACKs)
}

client := &Client{
Expand Down

0 comments on commit 5cd6c34

Please sign in to comment.