feat: fix bug on commit with RPC fail, no set failed flag for rollback invocation.

Author: KirillKurdyukov

src/Ydb.Sdk/src/Ado/YdbTransaction.cs

@@ -96,11 +96,15 @@ private async Task FinishTransaction(Func<string, Task<Status>> finishMethod)
 
             if (status.IsNotSuccess)
             {
+                Failed = true;
+
                 throw new YdbException(status);
             }
         }
         catch (Driver.TransportException e)
         {
+            Failed = true;
+
             throw new YdbException(e.Status);
         }
     }

src/Ydb.Sdk/src/Services/Query/SessionPool.cs

@@ -94,7 +94,7 @@ protected override async Task<Session> CreateSession()
             {
                 session.IsActive = false;
 
-                Logger.LogTrace("Attached stream is closed");
+                Logger.LogTrace("Session[{SessionId}]: Attached stream is closed", session.SessionId);
             }
         });
 

src/Ydb.Sdk/tests/Ado/YdbExceptionTests.cs

@@ -59,7 +59,7 @@ public async Task IsTransient_WhenAborted_ReturnTrueAndMakeEmptyRollback()
         Assert.Equal("This YdbTransaction has completed; it is no longer usable",
             Assert.Throws<InvalidOperationException>(() => ydbCommand.Transaction.Commit()).Message);
 
-        await ydbCommand.Transaction!.RollbackAsync();
+        await ydbCommand.Transaction.RollbackAsync();
         Assert.Equal("This YdbTransaction has completed; it is no longer usable",
             Assert.Throws<InvalidOperationException>(() => ydbCommand.Transaction!.Commit()).Message);
         Assert.Equal("This YdbTransaction has completed; it is no longer usable",