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
Streams update 2.4.16 #2710
Streams update 2.4.16 #2710
Conversation
{ | ||
var mat = Sys.Materializer(ActorMaterializerSettings.Create(Sys).WithAutoFusing(false)); | ||
var t = Source.Single(1) | ||
.ViaMaterialized(Flow.Identity<int>(), Keep.Both) |
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.
Should we change Keep.Both
to ValueTuple in the future?
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.
Good idea, I'll change it right away via nuget package.
/// TBD | ||
/// Materializer SPI (Service Provider Interface) | ||
/// | ||
/// Binary compatibility is NOT guaranteed on materializer internals. |
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.
We do not provide binary compatibility at the moment. It is redundant sentence
Have you implemented this one? |
We've reported this bug on the jvm and it was already implemented correctly so no need to port it. |
Once #2683 is merged those two changes need to ported too- https://github.com/akka/akka/pull/21989/files#diff-7104b3bccdee7b3ce9073c6f2179f42f- https://github.com/akka/akka/pull/21989/files#diff-3a67e6d52956dd56d88c954b5d90d201andasOutput stream close race
probably needs to be fixed, couldn't run the test while implemented it