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

Conversation

millems
Copy link
Contributor

@millems millems commented May 24, 2018

No description provided.

@millems millems requested a review from zoewangg May 24, 2018 23:44
Copy link
Contributor

@zoewangg zoewangg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

did the s3 integ tests succeed?

@@ -33,8 +33,8 @@
/**
* Handler context key for advanced configuration.
*/
public static final ExecutionAttribute<ServiceAdvancedConfiguration> SERVICE_ADVANCED_CONFIG =
new ExecutionAttribute<>("ServiceAdvancedConfig");
public static final ExecutionAttribute<ServiceConfiguration> SERVICE_ADVANCED_CONFIG =
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SERVICE_CONFIG

public static final ExecutionAttribute<ServiceAdvancedConfiguration> SERVICE_ADVANCED_CONFIG =
new ExecutionAttribute<>("ServiceAdvancedConfig");
public static final ExecutionAttribute<ServiceConfiguration> SERVICE_ADVANCED_CONFIG =
new ExecutionAttribute<>("ServiceServiceConfig");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two Services

@millems millems force-pushed the millem/service-config-rename branch 2 times, most recently from 88f61f4 to 6fad6b6 Compare May 25, 2018 20:34
@millems
Copy link
Contributor Author

millems commented May 25, 2018

Verified that the integration tests succeed.

@millems millems force-pushed the millem/service-config-rename branch from 6fad6b6 to dbdcdbb Compare May 25, 2018 20:52
@@ -0,0 +1,5 @@
{
"category": "AWS SDK for Java v2",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit, category should be Amazon S3 maybe?

Copy link
Contributor Author

@millems millems May 25, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed!

@millems millems force-pushed the millem/service-config-rename branch from dbdcdbb to 204a2e2 Compare May 25, 2018 21:10
@millems millems merged commit ad83616 into master May 25, 2018
@millems millems deleted the millem/service-config-rename branch May 25, 2018 21:22
aws-sdk-java-automation pushed a commit that referenced this pull request May 15, 2019
Revert "Standardize buildspec hacks around disabling maven connection pooling."
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants