Skip to content

OnExecutionFailure is not called when ApiCallTimeout occurs #2367

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

Closed
ndejaco2 opened this issue Apr 1, 2021 · 1 comment
Closed

OnExecutionFailure is not called when ApiCallTimeout occurs #2367

ndejaco2 opened this issue Apr 1, 2021 · 1 comment
Labels
bug This issue is a bug. closed-for-staleness duplicate This issue is a duplicate.

Comments

@ndejaco2
Copy link

ndejaco2 commented Apr 1, 2021

Describe the bug

I am attempting to use a custom execution interceptor to track concurrent executions on an async client. However, when the client call times out the onExecutionFailure callback is not made so the count of executions does not get decremented correctly.

Expected Behavior

When a call fails with api call timeout it should execute the onExecutionFailed callback in the interceptor.

Current Behavior

Currently, the callback is not executed.

Steps to Reproduce

Create a lambda function that will sleep for 5 seconds. Set api call timeout and apiCallAttemptTimeout to 1 second. Make a request with the client to invoke the function which should timeout. Verify that the callback is not executed.

Context

I am attempting to use a custom execution interceptor to track concurrent executions on an async client.

Your Environment

  • AWS Java SDK version used: latest
  • JDK version used: 11
  • Operating System and version: al2
@ndejaco2 ndejaco2 added bug This issue is a bug. needs-triage This issue or PR still needs to be triaged. labels Apr 1, 2021
@debora-ito
Copy link
Member

Hi @ndejaco2 I believe this is duplicate of #806. Let's keep track in that other issue, and please upvote the original description to show you are impacted by this. Apologies for taking this long to reply.

@debora-ito debora-ito added closing-soon This issue will close in 4 days unless further comments are made. duplicate This issue is a duplicate. and removed needs-triage This issue or PR still needs to be triaged. labels Apr 19, 2021
@github-actions github-actions bot added closed-for-staleness and removed closing-soon This issue will close in 4 days unless further comments are made. labels Apr 24, 2021
aws-sdk-java-automation added a commit that referenced this issue Feb 6, 2023
…c6675f2cc

Pull request: release <- staging/dec152f5-d20b-4254-b701-40dc6675f2cc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug. closed-for-staleness duplicate This issue is a duplicate.
Projects
None yet
Development

No branches or pull requests

2 participants