change default log for ProcessIncomingFrames thread (#275)

Gsantomaggio · web-flow · commit a9ee96a320a4 · 2023-06-13T09:29:32.000+02:00
* In a pressure system, the process frame could take longer to close.
* Increase a bit the wait time and change the log from Error to Warning because it is not an error, just a warning.
Signed-off-by: Gabriele Santomaggio &lt;G.santomaggio@gmail.com&gt;
diff --git a/RabbitMQ.Stream.Client/Connection.cs b/RabbitMQ.Stream.Client/Connection.cs
@@ -222,9 +222,9 @@ public void Dispose()
                     writer.Complete();
                     reader.Complete();
                     socket.Dispose();
-                    if (!_incomingFramesTask.Wait(Consts.ShortWait))
+                    if (!_incomingFramesTask.Wait(Consts.MidWait))
                     {
-                        _logger?.LogError("frame reader task did not exit in {ShortWait}", Consts.ShortWait);
+                        _logger?.LogWarning("ProcessIncomingFrames reader task did not exit in {MidWait}", Consts.MidWait);
                     }
                 }
                 finally

Original file line number	Diff line number	Diff line change
`@@ -222,9 +222,9 @@ public void Dispose()`
`222`	`222`	`writer.Complete();`
`223`	`223`	`reader.Complete();`
`224`	`224`	`socket.Dispose();`
`225`		`- if (!_incomingFramesTask.Wait(Consts.ShortWait))`
	`225`	`+ if (!_incomingFramesTask.Wait(Consts.MidWait))`
`226`	`226`	`{`
`227`		`- _logger?.LogError("frame reader task did not exit in {ShortWait}", Consts.ShortWait);`
	`227`	`+ _logger?.LogWarning("ProcessIncomingFrames reader task did not exit in {MidWait}", Consts.MidWait);`
`228`	`228`	`}`
`229`	`229`	`}`
`230`	`230`	`finally`