Reduce thread usage by removing a possible unneeded Task.Run

src/Speckle.Core/Transports/SQLite.cs

@@ -281,10 +281,7 @@ public override string ToString()
   /// Awaits untill write completion (ie, the current queue is fully consumed).
   /// </summary>
   /// <returns></returns>
-  public async Task WriteComplete()
-  {
-    await Utilities.WaitUntil(() => WriteCompletionStatus, 500).ConfigureAwait(false);
-  }
+  public async Task WriteComplete() => await Utilities.WaitUntil(() => WriteCompletionStatus, 500).ConfigureAwait(false);
 
   /// <summary>
   /// Returns true if the current write queue is empty and comitted.

src/Speckle.Core/Transports/Utilities.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Threading.Tasks;
-using Speckle.Core.Logging;
-
 namespace Speckle.Core.Transports;
 
 public static class Utilities
@@ -11,21 +7,12 @@ public static class Utilities
   /// </summary>
   /// <param name="condition"></param>
   /// <param name="frequency"></param>
-  /// <param name="timeout"></param>
   /// <returns></returns>
-  public static async Task WaitUntil(Func<bool> condition, int frequency = 25, int timeout = -1)
+  public static async Task WaitUntil(Func<bool> condition, int frequency = 25)
   {
-    var waitTask = Task.Run(async () =>
-    {
       while (!condition())
       {
         await Task.Delay(frequency).ConfigureAwait(false);
       }
-    });
-
-    if (waitTask != await Task.WhenAny(waitTask, Task.Delay(timeout)).ConfigureAwait(false))
-    {
-      throw new SpeckleException("Process timed out", new TimeoutException());
-    }
   }
 }

src/Speckle.Transports.MongoDB/MongoDB.cs

@@ -146,18 +146,13 @@ internal void DeleteObject(string hash)
   /// Awaits until write completion (ie, the current queue is fully consumed).
   /// </summary>
   /// <returns></returns>
-  public async Task WriteComplete()
-  {
+  public async Task WriteComplete() =>
     await Utilities
       .WaitUntil(
-        () =>
-        {
-          return GetWriteCompletionStatus();
-        },
+        GetWriteCompletionStatus,
         500
       )
       .ConfigureAwait(false);
-  }
 
   /// <summary>
   /// Returns true if the current write queue is empty and committed.