Add todos for business metrics

0marperez · 0marperez · commit d085dec970a2 · 2024-11-12T02:27:17.000-05:00
diff --git a/aws-runtime/aws-config/common/src/aws/sdk/kotlin/runtime/config/AbstractAwsSdkClientFactory.kt b/aws-runtime/aws-config/common/src/aws/sdk/kotlin/runtime/config/AbstractAwsSdkClientFactory.kt
@@ -98,9 +98,11 @@ public abstract class AbstractAwsSdkClientFactory<
             }
 
             if (config is HttpChecksumClientConfig.Builder) {
+                // TODO - business metric
                 config.requestChecksumCalculation =
                     config.requestChecksumCalculation ?: resolveRequestChecksumCalculation(platform, profile)
 
+                // TODO - business metric
                 config.responseChecksumValidation =
                     config.responseChecksumValidation ?: resolveResponseChecksumValidation(platform, profile)
             }

Original file line number	Diff line number	Diff line change
`@@ -98,9 +98,11 @@ public abstract class AbstractAwsSdkClientFactory<`
`98`	`98`	`}`
`99`	`99`
`100`	`100`	`if (config is HttpChecksumClientConfig.Builder) {`
	`101`	`+ // TODO - business metric`
`101`	`102`	`config.requestChecksumCalculation =`
`102`	`103`	`config.requestChecksumCalculation ?: resolveRequestChecksumCalculation(platform, profile)`
`103`	`104`
	`105`	`+ // TODO - business metric`
`104`	`106`	`config.responseChecksumValidation =`
`105`	`107`	`config.responseChecksumValidation ?: resolveResponseChecksumValidation(platform, profile)`
`106`	`108`	`}`