review feedback and misc fix

Author: Wraith2

src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlDataReader.cs

@@ -3533,7 +3533,7 @@ private TdsOperationStatus TryNextResult(out bool more)
                             if (result != TdsOperationStatus.Done)
                             {
                                 more = false;
-                                return TdsOperationStatus.Done;
+                                return result;
                             }
 
                             // In the case of not closing the reader, null out the metadata AFTER

src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/Packet.cs

@@ -3,6 +3,7 @@
 // See the LICENSE file in the project root for more information.
 
 using System;
+using System.Diagnostics;
 
 namespace Microsoft.Data.SqlClient
 {
@@ -101,7 +102,7 @@ public int RequiredLength
         }
 
         /// <summary>
-        /// returns a boolean value indicating if there are enough total bytes availble in the <see cref="Buffer"/> to read the tds header
+        /// returns a boolean value indicating if there are enough total bytes available in the <see cref="Buffer"/> to read the tds header
         /// </summary>
         public bool HasHeader => _totalLength >= TdsEnums.HEADER_LEN;
 
@@ -139,6 +140,7 @@ public void CheckDisposed()
             }
         }
 
+        [Conditional("DEBUG")]
         internal void SetCreatedBy(int creator) => SetCreatedByImpl(creator);
 
         partial void SetCreatedByImpl(int creator);

src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/TdsParserStateObject.cs

@@ -2086,7 +2086,7 @@ internal void ReadSniSyncOverAsync()
             }
 
             PacketHandle readPacket = default;
-            bool readFromNetwork = PartialPacketContainsCompletePacket();
+            bool readFromNetwork = !PartialPacketContainsCompletePacket();
             uint error;
 
             RuntimeHelpers.PrepareConstrainedRegions();