don't use *sync.Mutex when not necessary

Kong · Jan 7, 2025 · 2d33eeb · 2d33eeb
1 parent a912d99
commit 2d33eeb
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/internal/dataplane/sendconfig/dbmode.go b/internal/dataplane/sendconfig/dbmode.go
@@ -35,7 +35,7 @@ type UpdateStrategyDBMode struct {
 	isKonnect         bool
 	logger            logr.Logger
 	resourceErrors    []ResourceError
-	resourceErrorLock *sync.Mutex
+	resourceErrorLock sync.Mutex
 }
 
 // UpdateStrategyDBModeOpt is a functional option for UpdateStrategyDBMode.
@@ -57,13 +57,11 @@ func NewUpdateStrategyDBMode(
 	opts ...UpdateStrategyDBModeOpt,
 ) *UpdateStrategyDBMode {
 	s := &UpdateStrategyDBMode{
-		client:            client,
-		dumpConfig:        dumpConfig,
-		version:           version,
-		concurrency:       concurrency,
-		logger:            logger,
-		resourceErrors:    []ResourceError{},
-		resourceErrorLock: &sync.Mutex{},
+		client:      client,
+		dumpConfig:  dumpConfig,
+		version:     version,
+		concurrency: concurrency,
+		logger:      logger,
 	}
 	for _, opt := range opts {
 		opt(s)