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

Do not use expression based options for backoff props #6805

Conversation

to11mtm
Copy link
Member

@to11mtm to11mtm commented Jun 19, 2023

Fixes #6804. See #4698 for background.

Changes

Change BackoffOptions props to be non-expression based

Copy link
Member

@Aaronontheweb Aaronontheweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Aaronontheweb Aaronontheweb added the akka.net v1.4 Issues affecting Akka.NET v1.4 label Jun 19, 2023
@Aaronontheweb
Copy link
Member

We'll forward port this to v1.5 as well.

@Aaronontheweb Aaronontheweb enabled auto-merge (squash) June 19, 2023 20:00
@Aaronontheweb Aaronontheweb merged commit 931775a into akkadotnet:v1.4 Jun 19, 2023
13 checks passed
Aaronontheweb pushed a commit to Aaronontheweb/akka.net that referenced this pull request Jun 28, 2023
Aaronontheweb added a commit that referenced this pull request Jun 28, 2023
* Do not use expression based options for backoff props (#6805)

* Do not use expression based props for long lived streams (#6807)

* Do not use expression based props for long lived streams

* Update ActorMaterializerImpl.cs

---------

Co-authored-by: Aaron Stannard <aaron@petabridge.com>

---------

Co-authored-by: Drew <laingas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
akka.net v1.4 Issues affecting Akka.NET v1.4 perf
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants