Add GetAwaiter().GetResult(); on MessageHandler (#184)

* Change AwaitAsync to GetAwaiter().GetResult()

This should actually block until this handler executes.
Otherwise calling a few handlers eats up the whole threadpool

Per conversation with @ricsiLT

Signed-off-by: Gabriele Santomaggio <[email protected]>

Author: Gsantomaggio

RabbitMQ.Stream.Client/RawConsumer.cs

@@ -137,7 +137,7 @@ void DispatchMessage(ref SequenceReader<byte> sequenceReader, ulong i)
                     {
                         _config.MessageHandler(this,
                             new MessageContext(message.MessageOffset, TimeSpan.FromMilliseconds(chunk.Timestamp)),
-                            message);
+                            message).GetAwaiter().GetResult();
                     }
                 }
                 catch (Exception e)

RabbitMQ.Stream.Client/Reliable/Consumer.cs

@@ -37,7 +37,7 @@ public record ConsumerConfig : ReliableConfig
     // See also: https://blog.rabbitmq.com/posts/2022/07/rabbitmq-3-11-feature-preview-super-streams
     // Parameters:
     public bool IsSuperStream { get; set; }
-    
+
     // <summary>
     // The offset is the place in the stream where the consumer starts consuming from. The possible values for the offset parameter are the following:
     // - OffsetTypeFirst: starting from the first available offset. If the stream has not been truncated, this means the beginning of the stream (offset 0).
@@ -53,7 +53,7 @@ public record ConsumerConfig : ReliableConfig
     // The other instances will be idle.
     // </summary>
     public bool IsSingleActiveConsumer { get; set; } = false;
-    
+
     // <summary>
     // The broker notifies a consumer that becomes active before dispatching messages to it. 
     // With ConsumerUpdateListener the consumer can decide where to start consuming from.

RabbitMQ.Stream.Client/Reliable/Producer.cs

@@ -226,6 +226,11 @@ public async ValueTask Send(List<Message> messages, CompressionType compressionT
         }
     }
 
+    public override string ToString()
+    {
+        return $"Producer reference: {_producerConfig.Reference}, stream: {_producerConfig.Stream} ";
+    }
+
     /// <summary>
     /// Send the messages in batch to the stream in synchronous mode.
     /// The aggregation is provided by the user.
@@ -270,9 +275,4 @@ public async ValueTask Send(List<Message> messages)
             SemaphoreSlim.Release();
         }
     }
-
-    public override string ToString()
-    {
-        return $"Producer reference: {_producerConfig.Reference}, stream: {_producerConfig.Stream} ";
-    }
 }