Skip to content

Renamed S3AdvancedConfiguration to S3Configuration and AdvancedServiceConfiguration to ServiceConfiguration. #502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 25, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changes/next-release/feature-AWSSDKforJavav2-14a753c.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{
"category": "Amazon S3",
"type": "feature",
"description": "Renamed `S3AdvancedConfiguration` to `S3Configuration`"
}
5 changes: 5 additions & 0 deletions .changes/next-release/feature-AWSSDKforJavav2-7ac3210.json
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
{
"category": "AWS SDK for Java v2",
"type": "feature",
"description": "Renamed `AdvancedServiceConfiguration` to `ServiceConfiguration`"
}
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@
import software.amazon.awssdk.awscore.config.AwsAsyncClientConfiguration;
import software.amazon.awssdk.core.SdkRequest;
import software.amazon.awssdk.core.SdkResponse;
import software.amazon.awssdk.core.ServiceAdvancedConfiguration;
import software.amazon.awssdk.core.ServiceConfiguration;
import software.amazon.awssdk.core.async.AsyncResponseTransformer;
import software.amazon.awssdk.core.client.AsyncClientHandler;
import software.amazon.awssdk.core.client.BaseClientHandler;
Expand All @@ -36,7 +36,7 @@ public final class AwsAsyncClientHandler extends BaseClientHandler implements As

private final AsyncClientHandler delegateHandler;

public AwsAsyncClientHandler(AwsAsyncClientConfiguration clientConfiguration, ServiceAdvancedConfiguration
public AwsAsyncClientHandler(AwsAsyncClientConfiguration clientConfiguration, ServiceConfiguration
advancedClientOption) {
super(clientConfiguration, advancedClientOption);
this.delegateHandler = new AwsAsyncClientHandlerImpl(clientConfiguration, advancedClientOption);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
import software.amazon.awssdk.annotations.ThreadSafe;
import software.amazon.awssdk.awscore.config.AwsAsyncClientConfiguration;
import software.amazon.awssdk.core.SdkRequest;
import software.amazon.awssdk.core.ServiceAdvancedConfiguration;
import software.amazon.awssdk.core.ServiceConfiguration;
import software.amazon.awssdk.core.client.AsyncClientHandler;
import software.amazon.awssdk.core.client.BaseAsyncClientHandler;
import software.amazon.awssdk.core.http.AmazonAsyncHttpClient;
Expand All @@ -34,17 +34,17 @@
@SdkProtectedApi
final class AwsAsyncClientHandlerImpl extends BaseAsyncClientHandler {
private final AwsAsyncClientConfiguration clientConfiguration;
private final ServiceAdvancedConfiguration serviceAdvancedConfiguration;
private final ServiceConfiguration serviceConfiguration;

AwsAsyncClientHandlerImpl(AwsAsyncClientConfiguration clientConfiguration, ServiceAdvancedConfiguration
serviceAdvancedConfiguration) {
super(clientConfiguration, serviceAdvancedConfiguration, new AmazonAsyncHttpClient(clientConfiguration));
AwsAsyncClientHandlerImpl(AwsAsyncClientConfiguration clientConfiguration, ServiceConfiguration
serviceConfiguration) {
super(clientConfiguration, serviceConfiguration, new AmazonAsyncHttpClient(clientConfiguration));
this.clientConfiguration = clientConfiguration;
this.serviceAdvancedConfiguration = serviceAdvancedConfiguration;
this.serviceConfiguration = serviceConfiguration;
}

@Override
protected ExecutionContext createExecutionContext(SdkRequest originalRequest) {
return AwsClientHandlerUtils.createExecutionContext(originalRequest, clientConfiguration, serviceAdvancedConfiguration);
return AwsClientHandlerUtils.createExecutionContext(originalRequest, clientConfiguration, serviceConfiguration);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
import software.amazon.awssdk.awscore.config.AwsClientConfiguration;
import software.amazon.awssdk.core.RequestOverrideConfig;
import software.amazon.awssdk.core.SdkRequest;
import software.amazon.awssdk.core.ServiceAdvancedConfiguration;
import software.amazon.awssdk.core.ServiceConfiguration;
import software.amazon.awssdk.core.config.ClientOverrideConfiguration;
import software.amazon.awssdk.core.http.ExecutionContext;
import software.amazon.awssdk.core.interceptor.ExecutionAttributes;
Expand All @@ -41,7 +41,7 @@ private AwsClientHandlerUtils() {

static ExecutionContext createExecutionContext(SdkRequest originalRequest,
AwsClientConfiguration clientConfiguration,
ServiceAdvancedConfiguration serviceAdvancedConfiguration) {
ServiceConfiguration serviceConfiguration) {

AwsCredentialsProvider credentialsProvider = originalRequest.requestOverrideConfig()
.filter(c -> c instanceof AwsRequestOverrideConfig)
Expand All @@ -56,7 +56,7 @@ static ExecutionContext createExecutionContext(SdkRequest originalRequest,
Validate.validState(credentials != null, "Credential providers must never return null.");

ExecutionAttributes executionAttributes = new ExecutionAttributes()
.putAttribute(AwsExecutionAttributes.SERVICE_ADVANCED_CONFIG, serviceAdvancedConfiguration)
.putAttribute(AwsExecutionAttributes.SERVICE_CONFIG, serviceConfiguration)
.putAttribute(AwsExecutionAttributes.AWS_CREDENTIALS, credentials)
.putAttribute(AwsExecutionAttributes.REQUEST_CONFIG, originalRequest.requestOverrideConfig()
.map(c -> (RequestOverrideConfig) c)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
import software.amazon.awssdk.awscore.config.AwsSyncClientConfiguration;
import software.amazon.awssdk.core.SdkRequest;
import software.amazon.awssdk.core.SdkResponse;
import software.amazon.awssdk.core.ServiceAdvancedConfiguration;
import software.amazon.awssdk.core.ServiceConfiguration;
import software.amazon.awssdk.core.client.BaseClientHandler;
import software.amazon.awssdk.core.client.ClientExecutionParams;
import software.amazon.awssdk.core.client.SyncClientHandler;
Expand All @@ -35,10 +35,10 @@ public class AwsSyncClientHandler extends BaseClientHandler implements SyncClien

private final SyncClientHandler delegateHandler;

public AwsSyncClientHandler(AwsSyncClientConfiguration clientConfiguration, ServiceAdvancedConfiguration
serviceAdvancedConfiguration) {
super(clientConfiguration, serviceAdvancedConfiguration);
this.delegateHandler = new AwsSyncClientHandlerImpl(clientConfiguration, serviceAdvancedConfiguration);
public AwsSyncClientHandler(AwsSyncClientConfiguration clientConfiguration, ServiceConfiguration
serviceConfiguration) {
super(clientConfiguration, serviceConfiguration);
this.delegateHandler = new AwsSyncClientHandlerImpl(clientConfiguration, serviceConfiguration);
}

@Override
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
import software.amazon.awssdk.annotations.ThreadSafe;
import software.amazon.awssdk.awscore.config.AwsSyncClientConfiguration;
import software.amazon.awssdk.core.SdkRequest;
import software.amazon.awssdk.core.ServiceAdvancedConfiguration;
import software.amazon.awssdk.core.ServiceConfiguration;
import software.amazon.awssdk.core.client.BaseSyncClientHandler;
import software.amazon.awssdk.core.client.SyncClientHandler;
import software.amazon.awssdk.core.http.AmazonSyncHttpClient;
Expand All @@ -34,17 +34,17 @@
@SdkProtectedApi
final class AwsSyncClientHandlerImpl extends BaseSyncClientHandler {
private final AwsSyncClientConfiguration clientConfiguration;
private final ServiceAdvancedConfiguration serviceAdvancedConfiguration;
private final ServiceConfiguration serviceConfiguration;

AwsSyncClientHandlerImpl(AwsSyncClientConfiguration clientConfiguration, ServiceAdvancedConfiguration
serviceAdvancedConfiguration) {
super(clientConfiguration, serviceAdvancedConfiguration, new AmazonSyncHttpClient(clientConfiguration));
AwsSyncClientHandlerImpl(AwsSyncClientConfiguration clientConfiguration, ServiceConfiguration
serviceConfiguration) {
super(clientConfiguration, serviceConfiguration, new AmazonSyncHttpClient(clientConfiguration));
this.clientConfiguration = clientConfiguration;
this.serviceAdvancedConfiguration = serviceAdvancedConfiguration;
this.serviceConfiguration = serviceConfiguration;
}

@Override
protected ExecutionContext createExecutionContext(SdkRequest originalRequest) {
return AwsClientHandlerUtils.createExecutionContext(originalRequest, clientConfiguration, serviceAdvancedConfiguration);
return AwsClientHandlerUtils.createExecutionContext(originalRequest, clientConfiguration, serviceConfiguration);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -58,14 +58,14 @@ public TypeSpec poetSpec() {
}

private MethodSpec buildClientMethod() {
String advancedConfigParam = model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null ?
", advancedConfiguration()" : "";
String serviceConfigParam = model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null ?
", serviceConfiguration()" : "";
return MethodSpec.methodBuilder("buildClient")
.addAnnotation(Override.class)
.addModifiers(Modifier.PROTECTED, Modifier.FINAL)
.returns(clientInterfaceName)
.addCode("return new $T(super.asyncClientConfiguration() $L);",
clientClassName, advancedConfigParam)
clientClassName, serviceConfigParam)
.build();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,9 +64,9 @@ public TypeSpec poetSpec() {
ClassName.get(basePackage, model.getMetadata().getAsyncBuilder()));

if (model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null) {
ClassName advancedConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
builder.addField(FieldSpec.builder(advancedConfiguration, "advancedConfiguration")
ClassName serviceConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
builder.addField(FieldSpec.builder(serviceConfiguration, "serviceConfiguration")
.addModifiers(Modifier.PRIVATE)
.build());
}
Expand All @@ -76,9 +76,9 @@ public TypeSpec poetSpec() {
builder.addMethod(defaultSignerProviderMethod());

if (model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null) {
builder.addMethod(setAdvancedConfigurationMethod())
.addMethod(getAdvancedConfigurationMethod())
.addMethod(beanStyleSetAdvancedConfigurationMethod());
builder.addMethod(setServiceConfigurationMethod())
.addMethod(getServiceConfigurationMethod())
.addMethod(beanStyleSetServiceConfigurationMethod());
}

if (model.getCustomizationConfig().getServiceSpecificHttpConfig() != null) {
Expand Down Expand Up @@ -115,35 +115,35 @@ private MethodSpec serviceDefaultsMethod() {
.build();
}

private MethodSpec setAdvancedConfigurationMethod() {
ClassName advancedConfiguration = ClassName.get(basePackage,
private MethodSpec setServiceConfigurationMethod() {
ClassName serviceConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
return MethodSpec.methodBuilder("advancedConfiguration")
return MethodSpec.methodBuilder("serviceConfiguration")
.addModifiers(Modifier.PUBLIC)
.returns(TypeVariableName.get("B"))
.addParameter(advancedConfiguration, "advancedConfiguration")
.addStatement("this.advancedConfiguration = advancedConfiguration")
.addParameter(serviceConfiguration, "serviceConfiguration")
.addStatement("this.serviceConfiguration = serviceConfiguration")
.addStatement("return thisBuilder()")
.build();
}

private MethodSpec getAdvancedConfigurationMethod() {
ClassName advancedConfiguration = ClassName.get(basePackage,
private MethodSpec getServiceConfigurationMethod() {
ClassName serviceConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
return MethodSpec.methodBuilder("advancedConfiguration")
return MethodSpec.methodBuilder("serviceConfiguration")
.addModifiers(Modifier.PROTECTED)
.returns(advancedConfiguration)
.addStatement("return advancedConfiguration")
.returns(serviceConfiguration)
.addStatement("return serviceConfiguration")
.build();
}

private MethodSpec beanStyleSetAdvancedConfigurationMethod() {
ClassName advancedConfiguration = ClassName.get(basePackage,
private MethodSpec beanStyleSetServiceConfigurationMethod() {
ClassName serviceConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
return MethodSpec.methodBuilder("setAdvancedConfiguration")
return MethodSpec.methodBuilder("setServiceConfiguration")
.addModifiers(Modifier.PUBLIC)
.addParameter(advancedConfiguration, "advancedConfiguration")
.addStatement("advancedConfiguration(advancedConfiguration)")
.addParameter(serviceConfiguration, "serviceConfiguration")
.addStatement("serviceConfiguration(serviceConfiguration)")
.build();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,8 @@ public TypeSpec poetSpec() {
.addJavadoc(getJavadoc());

if (model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null) {
builder.addMethod(advancedConfigurationMethod());
builder.addMethod(advancedConfigurationConsumerBuilderMethod());
builder.addMethod(serviceConfigurationMethod());
builder.addMethod(serviceConfigurationConsumerBuilderMethod());
}

return builder.build();
Expand All @@ -64,27 +64,27 @@ private CodeBlock getJavadoc() {
ClassName.get(basePackage, model.getMetadata().getAsyncBuilderInterface()));
}

private MethodSpec advancedConfigurationMethod() {
ClassName advancedConfiguration = ClassName.get(basePackage,
private MethodSpec serviceConfigurationMethod() {
ClassName serviceConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
return MethodSpec.methodBuilder("advancedConfiguration")
return MethodSpec.methodBuilder("serviceConfiguration")
.addModifiers(Modifier.ABSTRACT, Modifier.PUBLIC)
.returns(TypeVariableName.get("B"))
.addParameter(advancedConfiguration, "advancedConfiguration")
.addParameter(serviceConfiguration, "serviceConfiguration")
.build();
}

private MethodSpec advancedConfigurationConsumerBuilderMethod() {
ClassName advancedConfiguration = ClassName.get(basePackage,
private MethodSpec serviceConfigurationConsumerBuilderMethod() {
ClassName serviceConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
TypeName consumerBuilder = ParameterizedTypeName.get(ClassName.get(Consumer.class),
advancedConfiguration.nestedClass("Builder"));
return MethodSpec.methodBuilder("advancedConfiguration")
serviceConfiguration.nestedClass("Builder"));
return MethodSpec.methodBuilder("serviceConfiguration")
.addModifiers(Modifier.DEFAULT, Modifier.PUBLIC)
.returns(TypeVariableName.get("B"))
.addParameter(consumerBuilder, "advancedConfiguration")
.addStatement("return advancedConfiguration($T.builder().apply(advancedConfiguration).build())",
advancedConfiguration)
.addParameter(consumerBuilder, "serviceConfiguration")
.addStatement("return serviceConfiguration($T.builder().apply(serviceConfiguration).build())",
serviceConfiguration)
.build();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,14 +59,14 @@ public TypeSpec poetSpec() {
}

private MethodSpec buildClientMethod() {
String advancedConfigParam = model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null ?
", advancedConfiguration()" : "";
String serviceConfigParam = model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null ?
", serviceConfiguration()" : "";
return MethodSpec.methodBuilder("buildClient")
.addAnnotation(Override.class)
.addModifiers(Modifier.PROTECTED, Modifier.FINAL)
.returns(clientInterfaceName)
.addCode("return new $T(super.syncClientConfiguration() $L);",
clientClassName, advancedConfigParam)
clientClassName, serviceConfigParam)
.build();
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ public TypeSpec poetSpec() {
.addMethod(protocolSpec.initProtocolFactory(model));

if (model.getCustomizationConfig().getServiceSpecificClientConfigClass() != null) {
classBuilder.addMethod(constructorWithAdvancedConfiguration());
classBuilder.addMethod(constructorWithServiceConfiguration());
} else {
classBuilder.addMethod(constructor());
}
Expand Down Expand Up @@ -97,13 +97,13 @@ private MethodSpec nameMethod() {
.build();
}

private MethodSpec constructorWithAdvancedConfiguration() {
ClassName advancedConfiguration = ClassName.get(basePackage,
private MethodSpec constructorWithServiceConfiguration() {
ClassName serviceConfiguration = ClassName.get(basePackage,
model.getCustomizationConfig().getServiceSpecificClientConfigClass());
return MethodSpec.constructorBuilder()
.addModifiers(Modifier.PROTECTED)
.addParameter(AwsAsyncClientConfiguration.class, "clientConfiguration")
.addParameter(advancedConfiguration, "serviceConfiguration")
.addParameter(serviceConfiguration, "serviceConfiguration")
.addStatement("this.clientHandler = new $T(clientConfiguration, serviceConfiguration)",
AwsAsyncClientHandler.class) // TODO this will likely differ for APIG clients
.addStatement("this.$N = init()", protocolSpec.protocolFactory(model).name)
Expand Down
Loading