-
Notifications
You must be signed in to change notification settings - Fork 916
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
Conversation
There was a problem hiding this 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 = |
There was a problem hiding this comment.
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"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
two Service
s
88f61f4
to
6fad6b6
Compare
Verified that the integration tests succeed. |
6fad6b6
to
dbdcdbb
Compare
@@ -0,0 +1,5 @@ | |||
{ | |||
"category": "AWS SDK for Java v2", |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed!
…dConfiguration to ServiceConfiguration.
dbdcdbb
to
204a2e2
Compare
Revert "Standardize buildspec hacks around disabling maven connection pooling."
No description provided.