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

[Kinesis customization] Read timeouts #134

Closed
Tracked by #724 ...
rcoh opened this issue Jan 13, 2021 · 1 comment
Closed
Tracked by #724 ...

[Kinesis customization] Read timeouts #134

rcoh opened this issue Jan 13, 2021 · 1 comment

Comments

@rcoh
Copy link
Collaborator

rcoh commented Jan 13, 2021

GetRecords requires a read timeout to prevent the HTTP client waiting forever. Medium because controlling read timeouts might be non-trivial

@rcoh rcoh changed the title [Kinesis customization] [Kinesis customization] Read timeouts Jan 13, 2021
@rcoh rcoh added the medium label Jan 13, 2021
@rcoh rcoh mentioned this issue Sep 27, 2021
6 tasks
@jdisanti jdisanti mentioned this issue Aug 31, 2022
28 tasks
@jdisanti
Copy link
Collaborator

Attempted to figure out the "why" for this issue. Found that it had a duplicate in smithy-kotlin that has since been resolved, and I think it originated from this Go SDK issue.

My stance on this is that since we're not going to implement a default read timeout, we don't need a customization to modify that default read timeout for Kinesis GetRecords.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants