-
Notifications
You must be signed in to change notification settings - Fork 647
Closed
Labels
feature-requestNew feature or enhancement. May require GitHub community feedback.New feature or enhancement. May require GitHub community feedback.
Description
Is your feature request related to a problem? Please describe.
Currently the constructor expects 3 individual variable, instead of one single hash. This is error prone and not aligned with other constructors in the SDK.
aws-sdk-js-v3/packages/middleware-retry/src/defaultStrategy.ts
Lines 37 to 41 in bcfa99c
| constructor( | |
| public readonly maxRetries: number, | |
| private retryDecider: RetryDecider = defaultRetryDecider, | |
| private delayDecider: DelayDecider = defaultDelayDecider | |
| ) {} |
Describe the solution you'd like
The parameter should be
export interface StrategyOption {
maxRetry: number,
private retryDecider?: RetryDecider,
private delayDecider?: DelayDecider
}/cc @trivikr
Metadata
Metadata
Assignees
Labels
feature-requestNew feature or enhancement. May require GitHub community feedback.New feature or enhancement. May require GitHub community feedback.