Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Conversation

@mxinden
Copy link
Contributor

@mxinden mxinden commented Aug 12, 2019

PeerSetHandle.set_priority_group allows modifying a priority group by
group identifier. With this commit the function can be accessed through
NetworkService.

This is need in order for a validator to connect to as many other
validators as configured without reserving a specific connection slot
for them.

This is needed for #3247, see #3247 (comment) for details.

`PeerSetHandle.set_priority_group` allows modifying a priority group by
group identifier. With this commit the function can be accessed through
`NetworkService`.

This is need in order for a validator to connect to as many other
validators as configured without reserving a specific connection slot
for them.
@mxinden mxinden added A0-please_review Pull request needs code review. M4-core Z1-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder labels Aug 12, 2019
@mxinden mxinden requested review from arkpar and tomaka August 12, 2019 16:40
@parity-cla-bot
Copy link

It looks like @mxinden signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

1 similar comment
@parity-cla-bot
Copy link

It looks like @mxinden signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@arkpar
Copy link
Member

arkpar commented Aug 12, 2019

Just a note: add_to_priority_group and remove_from_priority group might be useful as well.

@mxinden
Copy link
Contributor Author

mxinden commented Aug 12, 2019

Just a note: add_to_priority_group and remove_from_priority group might be useful as well.

As of right now I don't need those functions for #3247. Do you still want me to add them here @arkpar?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. Z1-easy Can be fixed primarily by duplicating and adapting code by an intermediate coder

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants