Skip to content

Commit

Permalink
Removed dev logs and comments.
Browse files Browse the repository at this point in the history
Signed-off-by: AWSHurneyt <hurneyt@amazon.com>
  • Loading branch information
AWSHurneyt committed Feb 5, 2024
1 parent a47fa02 commit f21379e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 14 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -579,8 +579,6 @@ data class Alert(
aggregationResultBucket?.innerXContent(builder)

if (!clusters.isNullOrEmpty()) builder.field(CLUSTERS_FIELD, clusters.toTypedArray())
// TODO hurneyt: Calling .toTypedArray() is required for a List<String> field to be successfully added to the alert document.
// Add a unit test that confirms each List<String>, and other lists, call .toTypedArray()

builder.endObject()
return builder
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,6 @@ data class ClusterMetricsInput(

// Verify parameters are valid during creation
init {
logger.info("hurneyt ClusterMetricsInput::clusters = $clusters")

require(validateFields()) {
"The uri.api_type field, uri.path field, or uri.uri field must be defined."
}
Expand All @@ -51,12 +49,7 @@ data class ClusterMetricsInput(

if (url.isNotEmpty() && validateFieldsNotEmpty()) {
require(constructedUri == constructUrlFromInputs()) {
"hurneyt The provided URL and URI fields form different URLs." +
"\nurl = $url " +
"\npath = $path " +
"\npathParams = $pathParams " +
"\nconstructedUri = $constructedUri " +
"\nconstructOutput = ${constructUrlFromInputs()}"
"The provided URL and URI fields form different URLs."
}
}

Expand Down Expand Up @@ -142,10 +135,7 @@ data class ClusterMetricsInput(
xcp.currentToken(),
xcp
)
while (xcp.nextToken() != XContentParser.Token.END_ARRAY) {
logger.info("hurneyt ClusterMetricsInput::parseInner xcp.text() = ${xcp.text()}")
clusters.add(xcp.text())
}
while (xcp.nextToken() != XContentParser.Token.END_ARRAY) clusters.add(xcp.text())
}
}
}
Expand Down

0 comments on commit f21379e

Please sign in to comment.