Browse Source

Fixed thread safety issue about KafkaOptions. #89

master
Savorboard 6 years ago
parent
commit
93804119b4
1 changed files with 9 additions and 7 deletions
  1. +9
    -7
      src/DotNetCore.CAP.Kafka/KafkaConsumerClient.cs

+ 9
- 7
src/DotNetCore.CAP.Kafka/KafkaConsumerClient.cs View File

@@ -66,16 +66,18 @@ namespace DotNetCore.CAP.Kafka


private void InitKafkaClient() private void InitKafkaClient()
{ {
_kafkaOptions.MainConfig["group.id"] = _groupId;
lock (_kafkaOptions)
{
_kafkaOptions.MainConfig["group.id"] = _groupId;


var config = _kafkaOptions.AsKafkaConfig();
_consumerClient = new Consumer<Null, string>(config, null, StringDeserializer);
_consumerClient.OnConsumeError += ConsumerClient_OnConsumeError;
_consumerClient.OnMessage += ConsumerClient_OnMessage;
_consumerClient.OnError += ConsumerClient_OnError;
var config = _kafkaOptions.AsKafkaConfig();
_consumerClient = new Consumer<Null, string>(config, null, StringDeserializer);
_consumerClient.OnConsumeError += ConsumerClient_OnConsumeError;
_consumerClient.OnMessage += ConsumerClient_OnMessage;
_consumerClient.OnError += ConsumerClient_OnError;
}
} }



private void ConsumerClient_OnConsumeError(object sender, Message e) private void ConsumerClient_OnConsumeError(object sender, Message e)
{ {
var message = e.Deserialize<Null, string>(null, StringDeserializer); var message = e.Deserialize<Null, string>(null, StringDeserializer);


Loading…
Cancel
Save