* Made session.Close() as nop for idled session

Author: asmyasnikov

CHANGELOG.md

@@ -1,4 +1,5 @@
-* Fixed closing of database/sql connection (aka `YDB` session)
+* Fixed closing of `database/sql` connection (aka `YDB` session)
+* Made `session.Close()` as `nop` for idled session
 
 ## v3.42.4
 * Added `ydb.WithDisableServerBalancer()` database/sql connector option

internal/table/session.go

@@ -184,17 +184,19 @@ func (s *session) Close(ctx context.Context) (err error) {
 
 		onDone := trace.TableOnSessionDelete(s.config.Trace(), &ctx, s)
 
-		_, err = s.tableService.DeleteSession(
-			ctx,
-			&Ydb_Table.DeleteSessionRequest{
-				SessionId: s.id,
-				OperationParams: operation.Params(ctx,
-					s.config.OperationTimeout(),
-					s.config.OperationCancelAfter(),
-					operation.ModeSync,
-				),
-			},
-		)
+		if time.Since(s.LastUsage()) < s.config.IdleThreshold() {
+			_, err = s.tableService.DeleteSession(
+				ctx,
+				&Ydb_Table.DeleteSessionRequest{
+					SessionId: s.id,
+					OperationParams: operation.Params(ctx,
+						s.config.OperationTimeout(),
+						s.config.OperationCancelAfter(),
+						operation.ModeSync,
+					),
+				},
+			)
+		}
 
 		for _, onClose := range s.onClose {
 			onClose(s)