extracting maxBatchSize number into a const with comment to explain i…

…ts use

aws/ecs_cluster_types.go

@@ -6,6 +6,13 @@ import (
 	"github.com/gruntwork-io/gruntwork-cli/errors"
 )
 
+// The pattern we use for running the `cloud-nuke` tool is to split the AWS API calls
+// into batches when the function `NukeAllResources` is executed.
+// A batch max number has been chosen for most modules.
+// However, for ECS clusters there is no explicit limiting described in the AWS CLI docs.
+// Therefore this `maxBatchSize` here is set to 100 as a safe maximum.
+const maxBatchSize = 100
+
 // ECSClusters - Represents all ECS clusters found in a region
 type ECSClusters struct {
 	ClusterArns []string
@@ -21,9 +28,8 @@ func (clusters ECSClusters) ResourceIdentifiers() []string {
 	return clusters.ClusterArns
 }
 
-// MaxBatchSize - the maximum number of ECS clusters for a single request
 func (clusters ECSClusters) MaxBatchSize() int {
-	return 200
+	return maxBatchSize
 }
 
 // Nuke - nuke all ECS Cluster resources