Skip to content

Conversation

@Kimax89
Copy link

@Kimax89 Kimax89 commented Aug 7, 2015

changed the maximum numbers of outbound connections from 8 to 16.
This is do to the general quality of the internet.

And i think this would benefit the network, and results in stronger connections from servers running the node.

changed the maximum numbers of outbound connection from 8 to 16.
This is do to the quality of the internet.
@Mirobit
Copy link
Contributor

Mirobit commented Aug 7, 2015

NACK, with a declining number of nodes we shouldn't put more pressure on them.
Not sure what you mean with quality of the internet. Number of nodes is the relevant factor here.

@rnicoll
Copy link
Contributor

rnicoll commented Aug 7, 2015

NACK at least without any analysis of impact on block propagation and bandwidth usage. Nodes relay (I believe) to all connections concurrently, so they'd be spreading upstream bandwidth even more thinly with this patch.

@maflcko
Copy link
Member

maflcko commented Aug 7, 2015

Can you provide evidence how this would benefit the network?

Also, there is a limited number of full nodes, thus a limited number of available inbound connections. What would happen, if there is a sudden demand (higher than the offered) number of inbound connections?

@fanquake
Copy link
Member

fanquake commented Aug 8, 2015

NACK See discussion in #6014 for background as to why increasing the max outbound connections to > 8 isn't a good idea.

@laanwj laanwj closed this Aug 10, 2015
@laanwj
Copy link
Member

laanwj commented Aug 10, 2015

NACK. I see no convincing evidence that this "benefits the network". At the very least you will have to precisely describe what you mean by "stronger connections". Stronger against what?

@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants