fixed race condition

KirillKurdyukov · KirillKurdyukov · commit ee578a2473ea · 2025-10-07T13:03:39.000+03:00
diff --git a/src/Ydb.Sdk/src/Ado/Session/ImplicitSessionSource.cs b/src/Ydb.Sdk/src/Ado/Session/ImplicitSessionSource.cs
@@ -38,7 +38,7 @@ private bool TryAcquireLease()
         if (Volatile.Read(ref _isDisposed) == 0)
             return true;
 
-        Interlocked.Decrement(ref _activeLeaseCount);
+        ReleaseLease();
         return false;
     }
 
diff --git a/src/Ydb.Sdk/test/Ydb.Sdk.Ado.Tests/Session/YdbImplicitStressTests.cs b/src/Ydb.Sdk/test/Ydb.Sdk.Ado.Tests/Session/YdbImplicitStressTests.cs
@@ -20,7 +20,7 @@ private IDriver DummyDriver()
     }
 
     [Fact]
-    public async Task Dispose_WaitsForAllLeases_AndSignalsOnEmptyExactlyOnce()
+    public async Task StressTest_OpenSession_RaceWithDispose_SuccessfulOpensAreNotDisposed()
     {
         for (var it = 0; it < 1000; it++)
         {

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ private bool TryAcquireLease()`
`38`	`38`	`if (Volatile.Read(ref _isDisposed) == 0)`
`39`	`39`	`return true;`
`40`	`40`
`41`		`- Interlocked.Decrement(ref _activeLeaseCount);`
	`41`	`+ ReleaseLease();`
`42`	`42`	`return false;`
`43`	`43`	`}`
`44`	`44`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ private IDriver DummyDriver()`
`20`	`20`	`}`
`21`	`21`
`22`	`22`	`[Fact]`
`23`		`- public async Task Dispose_WaitsForAllLeases_AndSignalsOnEmptyExactlyOnce()`
	`23`	`+ public async Task StressTest_OpenSession_RaceWithDispose_SuccessfulOpensAreNotDisposed()`
`24`	`24`	`{`
`25`	`25`	`for (var it = 0; it < 1000; it++)`
`26`	`26`	`{`