Skip to content

Commit

Permalink
Use CE connection args available, instead of hardcoded values (knativ…
Browse files Browse the repository at this point in the history
  • Loading branch information
aliok authored Apr 22, 2021
1 parent 7313634 commit a561719
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions pkg/channel/consolidated/dispatcher/dispatcher.go
Original file line number Diff line number Diff line change
Expand Up @@ -150,11 +150,8 @@ func NewDispatcher(ctx context.Context, args *KafkaDispatcherArgs) (*KafkaDispat
return nil, fmt.Errorf("unable to create kafka producer against Kafka bootstrap servers %v : %v", args.Brokers, err)
}

// Configured with the same connection arguments of IMC
kncloudevents.ConfigureConnectionArgs(&kncloudevents.ConnectionArgs{
MaxIdleConns: 1000,
MaxIdleConnsPerHost: 100,
})
// Configure connection arguments
kncloudevents.ConfigureConnectionArgs(args.KnCEConnectionArgs)

dispatcher := &KafkaDispatcher{
dispatcher: eventingchannels.NewMessageDispatcher(logging.FromContext(ctx).Desugar()),
Expand Down

0 comments on commit a561719

Please sign in to comment.