Skip to content
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

[fix] [SageMakerRuntime] empty body string when invokeEndpointAsync #4204

Closed
wants to merge 1 commit into from

Conversation

tan-t
Copy link

@tan-t tan-t commented Sep 10, 2022

what I did

  • replace the request body with an empty string when calling Async Inference.

related issue

#4203

Checklist
  • npm run test passes
  • changelog is added, npm run add-change

@tan-t
Copy link
Author

tan-t commented Nov 16, 2022

@aws-sdk-js-team

hi!
it looks like that this PR has been left unreviewed yet.
is everything ok? how can I help?
thanks in advance!

@aBurmeseDev
Copy link
Member

Hi @tan-t - I understand it's been a long silence here and my apologies for that.

I recently brought this up to team discussion and upon reviewing the issue, it seems like you've already found a workaround. I know it's not ideal but due to our limited resources to repro, review and test code changes, we would not be planning to accept this PR unfortunately.

If anyone runs into this, please refer to the workaround mentioned (#4203 (comment)).
Thank you for your understanding!

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.

None yet

2 participants