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

@@ -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,