eliminate a thread switch when invoking server-side handler

ojh3636 · May 9, 2016 · 65ca9dc · 65ca9dc
1 parent 92a9156
commit 65ca9dc
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/csharp/Grpc.Core/Server.cs b/src/csharp/Grpc.Core/Server.cs
@@ -283,18 +283,18 @@ private async Task HandleCallAsync(ServerRpcNew newRpc)
         /// </summary>
         private void HandleNewServerRpc(bool success, BatchContextSafeHandle ctx)
         {
+            Task.Run(AllowOneRpc);
+
             if (success)
             {
                 ServerRpcNew newRpc = ctx.GetServerRpcNew(this);
 
                 // after server shutdown, the callback returns with null call
                 if (!newRpc.Call.IsInvalid)
                 {
-                    Task.Run(async () => await HandleCallAsync(newRpc)).ConfigureAwait(false);
+                    HandleCallAsync(newRpc);  // we don't need to await.
                 }
             }
-
-            AllowOneRpc();
         }
 
         /// <summary>