Skip to content

Conversation

@jonasschnelli
Copy link
Contributor

We have seen node operators shutting down nodes because of traffic exceeding. Reducing traffic is possible with different measures and therefore very likely a "communication problem". This documentation might help those who like to or need to reduce traffic.

Thanks @fanquake for the overhaul!

@jonasschnelli
Copy link
Contributor Author

Release notes for maxuploadtarget is handled over #6958.

@maflcko
Copy link
Member

maflcko commented Nov 11, 2015

Looks good. Concept ACK

Copy link
Member

Choose a reason for hiding this comment

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

Nit: No CAPSLOCK

@jonasschnelli
Copy link
Contributor Author

Fixed @MarcoFalke's nits. The bold test part is okay IMO.

Copy link
Contributor

Choose a reason for hiding this comment

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

  • bitcoin's
  • handful

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! Force-Push fixed.

@jtimon
Copy link
Contributor

jtimon commented Nov 11, 2015

ACK

@jmcorgan
Copy link
Contributor

utACK

@gavinandresen
Copy link
Contributor

ACK.

(Nit: I don't like the use of the passive voice; for example "Reducing the maximum connected nodes to a miniumum could be desirable" would be clearer as "Set -maxconnections=1 for minimal bandwidth usage" ... but I think this is good enough to pull as-is).

@laanwj laanwj merged commit 42bae94 into bitcoin:master Nov 11, 2015
laanwj added a commit that referenced this pull request Nov 11, 2015
42bae94 [doc] Improve lanaguge in reducetraffic.md (fanquake)
6a776fa [doc] add documentation how to reduce traffic (Jonas Schnelli)
zkbot added a commit to zcash/zcash that referenced this pull request Feb 18, 2021
@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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants