Skip to content
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 9 additions & 7 deletions RELEASE_NOTES.md
Original file line number Diff line number Diff line change
@@ -1,15 +1,17 @@
# Frequenz Client Base Library Release Notes

## Upgrading
## Summary

* There is very minor breaking change in this release, `GrpcStreamBroadcaster` now requires a `grpc.aio.UnaryStreamCall` instead of a `AsyncIterable` for the `stream_method` argument. In practice all users should be passing a `grpc.aio.UnaryStreamCall` already, so this should not affect anyone unless they are doing something very strange.
<!-- Here goes a general summary of what this release is about -->

## Bug Fixes
## Upgrading

* `GrpcStreamBroadcaster`: Fix potential long delays on retries, and giving up early if the number of retries is limited.
<!-- Here goes notes on how to upgrade from previous versions, including deprecations and what they should be replaced with -->

The retry strategy was not reset after a successful start of the stream, so the back-off delays would accumulate over multiple retries and eventually give up if the number of retries were limited, even if there was a successful start of the stream in between. Now we properly reset the retry strategy after a successful start of the stream (successfully receiving the first item from the stream).
## New Features

* `GrpcStreamBroadcaster`: Fix `StreamStarted` event firing too soon.
<!-- Here goes the main new features and examples or instructions on how to use them -->

## Bug Fixes

The `StreamStarted` event was being fired as soon as the streamming method was called, but that doesn't mean that a streamming connection was established with the server at all, which can give a false impression that the stream is active and working. Now we wait until we receive the initial metadata from the server before firing the `StreamStarted` event. That should give users a better indication that the stream is actually active and working without having to wait for the first item to be received, which can take a long time for some low-frequency streams.
<!-- Here goes notable bug fixes that are worth a special mention or explanation -->