Skip to content

Commit

Permalink
Remove unnecessary log scoping in some physical backends
Browse files Browse the repository at this point in the history
  • Loading branch information
jefferai committed Jul 13, 2018
1 parent 9da2de1 commit f85efad
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion physical/dynamodb/dynamodb.go
Expand Up @@ -243,7 +243,7 @@ func NewDynamoDBBackend(conf map[string]string, logger log.Logger) (physical.Bac
return nil, errwrap.Wrapf("failed parsing max_parallel parameter: {{err}}", err)
}
if logger.IsDebug() {
logger.Debug("physical/dynamodb: max_parallel set", "max_parallel", maxParInt)
logger.Debug("max_parallel set", "max_parallel", maxParInt)
}
}

Expand Down
6 changes: 3 additions & 3 deletions physical/gcs/gcs.go
Expand Up @@ -86,7 +86,7 @@ type Backend struct {
// specifying credentials via envvars, credential files, etc. from environment
// variables or a service account file
func NewBackend(c map[string]string, logger log.Logger) (physical.Backend, error) {
logger.Debug("physical/gcs: configuring backend")
logger.Debug("configuring backend")

// Bucket name
bucket := os.Getenv(envBucket)
Expand Down Expand Up @@ -133,13 +133,13 @@ func NewBackend(c map[string]string, logger log.Logger) (physical.Backend, error
return nil, errwrap.Wrapf("failed to parse max_parallel: {{err}}", err)
}

logger.Debug("physical/gcs: configuration",
logger.Debug("configuration",
"bucket", bucket,
"chunk_size", chunkSize,
"ha_enabled", haEnabled,
"max_parallel", maxParallel,
)
logger.Debug("physical/gcs: creating client")
logger.Debug("creating client")

// Client
opts := []option.ClientOption{option.WithUserAgent(useragent.String())}
Expand Down

0 comments on commit f85efad

Please sign in to comment.