-
Notifications
You must be signed in to change notification settings - Fork 47
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: remove content_sent from StreamingSequences #1351
Conversation
Please run |
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.
Can you include a little bit more detail the PR description? It's a tad vague how this get's closer to something (which is undefined in the description)
updated description |
🤖 I have created a release *beep* *boop* --- ## [0.28.3](https://togithub.com/googleapis/gapic-showcase/compare/v0.28.2...v0.28.3) (2023-07-07) ### Bug Fixes * Remove content_sent from StreamingSequences ([#1351](https://togithub.com/googleapis/gapic-showcase/issues/1351)) ([ae910cc](https://togithub.com/googleapis/gapic-showcase/commit/ae910cca182787fb5babbfe1005b086d4f2f8c4b)) --- This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Currently, server streaming APIs do not resume from the point of failure. Before removing the contentSent property, the server would keep track of the last content that was sent before failing. To get closure to how server streams currently react to failure, we removed the contentSent property. This means that when we retry the request, the server will send the entire content, rather than starting from where it left off.