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.

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

Remove inOwnerId and outOwnerId from Connection #22447

Closed
drewhk opened this issue Mar 3, 2017 · 0 comments
Closed

Remove inOwnerId and outOwnerId from Connection #22447

drewhk opened this issue Mar 3, 2017 · 0 comments
Assignees
@johanandren
Labels
t:stream
Milestone
2.5.0-RC1

Comments

@drewhk
Copy link
Member

drewhk commented Mar 3, 2017

These fields are only used to access the open connections counter array, but it is simpler to just store the counter in the GraphStageLogic instead and remove these fields. This will simplify things quite a bit and might be even somewhat faster.
@drewhk drewhk added 1 - triaged Tickets that are safe to pick up for contributing in terms of likeliness of being accepted new-materializer t:stream labels Mar 3, 2017
@drewhk drewhk added this to the 2.5.0-RC1 milestone Mar 3, 2017
@johanandren johanandren added 3 - in progress Someone is working on this ticket and removed 1 - triaged Tickets that are safe to pick up for contributing in terms of likeliness of being accepted labels Mar 10, 2017
@johanandren johanandren self-assigned this Mar 10, 2017
johanandren added a commit to johanandren/akka that referenced this issue Mar 10, 2017
@johanandren
Remove superflous ownerId fields in interpreter Connection akka#22447
41e13ad
@johanandren johanandren mentioned this issue Mar 10, 2017
Merged
ktoso pushed a commit that referenced this issue Mar 13, 2017
@johanandren @ktoso
Remove superflous ownerId fields in interpreter Connection #22447
5ea2e05
@ktoso ktoso removed the 3 - in progress Someone is working on this ticket label Mar 13, 2017
@ktoso ktoso closed this as completed Mar 13, 2017
@alexvaluyskiy alexvaluyskiy mentioned this issue Sep 8, 2017
Closed
21 tasks
@marcpiechura marcpiechura mentioned this issue Feb 9, 2018
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@johanandren johanandren

Labels
t:stream
Projects
None yet
Milestone
2.5.0-RC1
Development

No branches or pull requests
3 participants
@ktoso @johanandren @drewhk