Skip to content

Don't leak sub flows when mat fails #22647

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 3, 2017
Merged

Don't leak sub flows when mat fails #22647

merged 1 commit into from
Apr 3, 2017

Conversation

johanandren
Copy link
Contributor

Some I fixed, some I just added test coverage for, but were already behaving.

Refs #22646

@akka-ci akka-ci added the validating PR is currently being validated by Jenkins label Mar 28, 2017
@johanandren
Copy link
Contributor Author

Will need forward port to 2.5

@akka-ci akka-ci added tested PR that was successfully built and tested by Jenkins and removed validating PR is currently being validated by Jenkins labels Mar 28, 2017
@akka-ci
Copy link

akka-ci commented Mar 28, 2017

Test PASSed.

ktoso
ktoso approved these changes Mar 30, 2017
View reviewed changes
Copy link
Contributor

@ktoso ktoso left a comment

Choose a reason for hiding this comment

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

LGTM

patriknw
patriknw approved these changes Apr 3, 2017
View reviewed changes
Copy link
Contributor

@patriknw patriknw left a comment

Choose a reason for hiding this comment

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

LGTM

@johanandren johanandren merged commit b6aec07 into akka:release-2.4 Apr 3, 2017
@johanandren johanandren deleted the wip-22646-graph-interpreter-leak-subflow-johanandren branch April 3, 2017 13:37
@Arkatufus Arkatufus mentioned this pull request Jun 5, 2017
Closed
Arkatufus added a commit to Arkatufus/akka.net that referenced this pull request Jun 6, 2017
@Arkatufus
Port akka/akka#22647
792657f
@Arkatufus Arkatufus mentioned this pull request Jun 6, 2017
Closed
marcpiechura pushed a commit to marcpiechura/akka.net that referenced this pull request Jun 7, 2017
@Arkatufus @marcpiechura
Port akka/akka#22647
dd2e3bd
@Arkatufus Arkatufus mentioned this pull request Jun 8, 2017
Merged
@alexvaluyskiy alexvaluyskiy mentioned this pull request Jun 10, 2017
Closed
21 tasks
Aaronontheweb pushed a commit to Aaronontheweb/akka.net that referenced this pull request Jun 28, 2017
@Arkatufus @Aaronontheweb
Port akka/akka#22647
b60ca0b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@patriknw patriknw patriknw approved these changes

+1 more reviewer

@ktoso ktoso ktoso approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
tested PR that was successfully built and tested by Jenkins
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@johanandren @akka-ci @ktoso @patriknw