From 2da4743179226133d85781faeeb2fcd3ee65e1e4 Mon Sep 17 00:00:00 2001
From: Rodrigo Zhou <rodrigozhou@users.noreply.github.com>
Date: Thu, 10 Aug 2023 16:03:46 -0500
Subject: [PATCH] Remove debug logger from visibility manager (#4751)

<!-- Describe what has changed in this PR -->
**What changed?**
Remove debug logger from visibility manager.

<!-- Tell your future self why have you made these changes -->
**Why?**
It can print DB username and password.

<!-- How have you verified this change? Tested locally? Added a unit
test? Checked in staging env? -->
**How did you test it?**


<!-- Assuming the worst case, what can be broken when deploying this
change to production? -->
**Potential risks**
No.

<!-- Is this PR a hotfix candidate or require that a notification be
sent to the broader community? (Yes/No) -->
**Is hotfix candidate?**
No.
---
 common/persistence/visibility/factory.go | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/common/persistence/visibility/factory.go b/common/persistence/visibility/factory.go
index c89ae50b421..48898151898 100644
--- a/common/persistence/visibility/factory.go
+++ b/common/persistence/visibility/factory.go
@@ -28,7 +28,6 @@ import (
 	"go.temporal.io/server/common/config"
 	"go.temporal.io/server/common/dynamicconfig"
 	"go.temporal.io/server/common/log"
-	"go.temporal.io/server/common/log/tag"
 	"go.temporal.io/server/common/metrics"
 	"go.temporal.io/server/common/persistence/sql/sqlplugin/mysql"
 	"go.temporal.io/server/common/persistence/sql/sqlplugin/postgresql"
@@ -65,9 +64,6 @@ func NewManager(
 	metricsHandler metrics.Handler,
 	logger log.Logger,
 ) (manager.VisibilityManager, error) {
-	// Log configuration
-	logger.Info("loading new manager with the configuration: ", tag.Key("VisibilityStoreConfig"), tag.Value(persistenceCfg.GetVisibilityStoreConfig()))
-
 	visibilityManager, err := newVisibilityManagerFromDataStoreConfig(
 		persistenceCfg.GetVisibilityStoreConfig(),
 		persistenceResolver,