refactor internal/table/client and upload with off gocognit linter

Author: grishagavrin

CHANGELOG.md

@@ -1,3 +1,4 @@
+* Refactored `internal/table/client.go` and extract func `client.onCloseSession`
 * Refactored `internal/decimal/decimal.go` and extract func `decimal.dotStringAnalysis`
 * Refactored `internal/backoff/backoff_test.go` and extract func `backoff_test.checkExpWithAct` for compare
 * Refactored `internal/xsql/dsn.go` and extract func `dsn.bindTablePathPrefixInConnectorOptions`

internal/table/client.go

@@ -173,21 +173,7 @@ func (c *Client) createSession(ctx context.Context, opts ...createSessionOption)
 					defer cancel()
 				}
 
-				closeSession := func(s *session) {
-					if s == nil {
-						return
-					}
-
-					closeSessionCtx := xcontext.WithoutDeadline(ctx)
-
-					if timeout := c.config.DeleteTimeout(); timeout > 0 {
-						var cancel context.CancelFunc
-						createSessionCtx, cancel = xcontext.WithTimeout(closeSessionCtx, timeout)
-						defer cancel()
-					}
-
-					_ = s.Close(closeSessionCtx)
-				}
+				closeSession := onCloseSession(ctx, createSessionCtx, c)
 
 				s, err = c.build(createSessionCtx)
 
@@ -223,6 +209,25 @@ func (c *Client) createSession(ctx context.Context, opts ...createSessionOption)
 	}
 }
 
+// onCloseSession is a closure function that takes a session and performs the actions to close it.
+func onCloseSession(ctx, createSessionCtx context.Context, c *Client) func(s *session) {
+	return func(s *session) {
+		if s == nil {
+			return
+		}
+
+		closeSessionCtx := xcontext.WithoutDeadline(ctx)
+
+		if timeout := c.config.DeleteTimeout(); timeout > 0 {
+			var cancel context.CancelFunc
+			createSessionCtx, cancel = xcontext.WithTimeout(closeSessionCtx, timeout)
+			defer cancel()
+		}
+
+		_ = s.Close(closeSessionCtx)
+	}
+}
+
 func (c *Client) CreateSession(ctx context.Context, opts ...table.Option) (_ table.ClosableSession, err error) {
 	if c == nil {
 		return nil, xerrors.WithStackTrace(errNilClient)