Skip to content

Commit 9198155

Browse files
committed
Remove a comment that might be confusing
1 parent 2747981 commit 9198155

File tree

1 file changed

+0
-3
lines changed

1 file changed

+0
-3
lines changed

codegen/src/main/java/software/amazon/awssdk/codegen/poet/client/ClientClassUtils.java

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -271,9 +271,6 @@ public static MethodSpec updateRetryStrategyClientConfigurationMethod() {
271271
.addStatement("configuration.option($T.RETRY_STRATEGY, defaultBuilder.build())", SdkClientOption.class)
272272
.addStatement("return")
273273
.endControlFlow();
274-
// This might be just blindly setting the same strategy that was already configured instead of a given override, but we
275-
// don't have any means to know if the retryStrategy was overridden, that's OK compared to having to expand the API to
276-
// add a method for it, such as overriddenRetryStrategy().
277274
builder.addStatement("$T retryStrategy = builder.retryStrategy()", RetryStrategy.class);
278275
builder.beginControlFlow("if (retryStrategy != null)")
279276
.addStatement("configuration.option($T.RETRY_STRATEGY, retryStrategy)", SdkClientOption.class)

0 commit comments

Comments
 (0)