Fix linter warning for batch workflow/CLI

MartinGomes · Jul 23, 2019 · 25d7054 · 25d7054
1 parent 8771d23
commit 25d7054
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/service/worker/batcher/workflow.go b/service/worker/batcher/workflow.go
@@ -48,10 +48,13 @@ const (
 	InfiniteDuration = 20 * 365 * 24 * time.Hour
 	pageSize         = 1000
 
-	// below are default values for BatchParams
-	DefaultRPS                      = 50
-	DefaultConcurrency              = 5
+	// DefaultRPS is the default RPS
+	DefaultRPS = 50
+	// DefaultConcurrency is the default concurrency
+	DefaultConcurrency = 5
+	// DefaultAttemptsOnRetryableError is the default value for AttemptsOnRetryableError
 	DefaultAttemptsOnRetryableError = 50
+	// DefaultActivityHeartBeatTimeout is the default value for ActivityHeartBeatTimeout
 	DefaultActivityHeartBeatTimeout = time.Second * 10
 )
 

diff --git a/tools/cli/workflowBatchCommands.go b/tools/cli/workflowBatchCommands.go
@@ -124,7 +124,7 @@ func ListBatchJobs(c *cli.Context) {
 	prettyPrintJSONObject(output)
 }
 
-// DescribeBatchJob describe the status of the batch job
+// StartBatchJob starts a batch job
 func StartBatchJob(c *cli.Context) {
 	domain := getRequiredGlobalOption(c, FlagDomain)
 	query := getRequiredOption(c, FlagListQuery)