5 Commits (master)

Author	SHA1	Message	Date
Martin HS	9045b79bc2	metrics, cmd/geth: change init-process of metrics (#30814) ... This PR modifies how the metrics library handles `Enabled`: previously, the package `init` decided whether to serve real metrics or just dummy-types. This has several drawbacks: - During pkg init, we need to determine whether metrics are enabled or not. So we first hacked in a check if certain geth-specific commandline-flags were enabled. Then we added a similar check for geth-env-vars. Then we almost added a very elaborate check for toml-config-file, plus toml parsing. - Using "real" types and dummy types interchangeably means that everything is hidden behind interfaces. This has a performance penalty, and also it just adds a lot of code. This PR removes the interface stuff, uses concrete types, and allows for the setting of Enabled to happen later. It is still assumed that `metrics.Enable()` is invoked early on. The somewhat 'heavy' operations, such as ticking meters and exp-decay, now checks the enable-flag to prevent resource leak. The change may be large, but it's mostly pretty trivial, and from the last time I gutted the metrics, I ensured that we have fairly good test coverage. --------- Co-authored-by: Felix Lange <fjl@twurst.com>	2 months ago
Justin Traglia	2c5648d891	all: fix some typos (#25551) ... * Fix some typos * Fix some mistakes * Revert 4byte.json * Fix an incorrect fix * Change files to fails	2 years ago
Péter Szilágyi	ff3535e8e0	p2p/tracker: only reschedule wake if previous didn't run	4 years ago
Martin Holst Swende	45fca44c24	p2p/tracker: properly clean up fulfilled requests	4 years ago
Péter Szilágyi	1fb9a6dd32	eth/protocols, prp/tracker: add support for req/rep rtt tracking (#22608) ... * eth/protocols, prp/tracker: add support for req/rep rtt tracking * p2p/tracker: sanity cap the number of pending requests * pap/tracker: linter <3 * p2p/tracker: disable entire tracker if no metrics are enabled	4 years ago