update socket lib

BasicTest/Program.cs

 using System;
 using System.Diagnostics;
+using System.Threading;
 using System.Threading.Tasks;
 using StackExchange.Redis;
 
@@ -9,6 +10,7 @@ internal static class Program
     {
         public static async Task Main()
         {
+            Thread.CurrentThread.Name = nameof(Main);
             using (var conn = await ConnectionMultiplexer.ConnectAsync("127.0.0.1:6379,syncTimeout=2000"))
             {
                 int expected = 0;

StackExchange.Redis/StackExchange.Redis.csproj

@@ -31,6 +31,6 @@
     <PackageReference Include="System.Memory" Version="$(CoreFxVersion)" />
     <PackageReference Include="System.Buffers" Version="$(CoreFxVersion)" />
     <PackageReference Include="System.IO.Pipelines" Version="$(CoreFxVersion)" />
-    <PackageReference Include="Pipelines.Sockets.Unofficial" Version="0.2.0-alpha-005" />
+    <PackageReference Include="Pipelines.Sockets.Unofficial" Version="0.2.0-alpha-007" />
   </ItemGroup>
 </Project>
\ No newline at end of file

StackExchange.Redis/StackExchange/Redis/PhysicalConnection.cs

@@ -975,27 +975,34 @@ void ISocketCallback.OnHeartbeat()
         }
 
         partial void OnWrapForLogging(ref IDuplexPipe pipe, string name);
-
         private async void ReadFromPipe() // yes it is an async void; deal with it!
         {
             try
             {
+                bool allowSyncRead = true;
                 while (true)
                 {
                     var input = _ioPipe?.Input;
                     if (input == null) break;
 
-                    var readResult = await input.ReadAsync();
+                    // note: TryRead will give us back the same buffer in a tight loop
+                    // - so: only use that if we're making progress
+                    if(!(allowSyncRead && input.TryRead(out var readResult)))
+                    {
+                        readResult = await input.ReadAsync();
+                    }
+                    
                     if (readResult.IsCompleted && readResult.Buffer.IsEmpty)
                     {
                         break; // we're all done
                     }
                     var buffer = readResult.Buffer;
-    
+
                     var s = new RawResult(ResultType.BulkString, buffer, false).GetString().Replace("\r","\\r").Replace("\n","\\n");
-                    
+
                     int handled = ProcessBuffer(ref buffer); // updates buffer.Start
-                    
+                    allowSyncRead = handled != 0;
+
                     Multiplexer.Trace($"Processed {handled} messages", physicalName);
                     input.AdvanceTo(buffer.Start, buffer.End);
                 }

StackExchange.Redis/StackExchange/Redis/SocketManager.cs

@@ -174,8 +174,12 @@ internal SocketToken BeginConnect(EndPoint endpoint, ISocketCallback callback, C
             {
                 var formattedEndpoint = Format.ToString(endpoint);
 
-                SocketConnection.ConnectAsync(endpoint, PipeOptions,
-                    conn => EndConnectAsync(conn, multiplexer, log, callback), socket);
+                var t = SocketConnection.ConnectAsync(endpoint, PipeOptions,
+                    //SocketConnectionOptions.SyncReader | SocketConnectionOptions.SyncWriter,
+                    SocketConnectionOptions.None,
+                    onConnected: conn => EndConnectAsync(conn, multiplexer, log, callback),
+                    socket: socket);
+                GC.KeepAlive(t); // make compiler happier
             }
             catch (NotImplementedException ex)
             {