Merge pull request #3117 from jaredpar/server

Don't restart on failed connection sequence

src/Compilers/Core/VBCSCompiler/ServerDispatcher.cs

@@ -403,7 +403,7 @@ internal static async Task<ConnectionData> CreateHandleConnectionTask(Task<Named
                 // Unable to establish a connection with the client.  The client is responsible for
                 // handling this case.  Nothing else for us to do here.
                 CompilerServerLogger.LogException(ex, "Error creating client named pipe");
-                return new ConnectionData(CompletionReason.ClientDisconnect);
+                return new ConnectionData(CompletionReason.CompilationNotStarted);
             }
 
             return await connection.ServeConnection(cancellationToken).ConfigureAwait(false);

src/Compilers/Core/VBCSCompilerTests/CompilerServerApiTest.cs

@@ -267,7 +267,7 @@ public async Task FailedConnectionShoudlCreateFailedConnectionData()
 
             tcs.SetException(new Exception());
             var connectionData = await connectionDataTask.ConfigureAwait(false);
-            Assert.Equal(ServerDispatcher.CompletionReason.ClientDisconnect, connectionData.CompletionReason);
+            Assert.Equal(ServerDispatcher.CompletionReason.CompilationNotStarted, connectionData.CompletionReason);
             Assert.Null(connectionData.KeepAlive);
         }