ElementsProject / ElementsProject/elements

Liquid INV rate probably too low

Offen
#1,243 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
C++
Sterne
1.2k
Forks
416
Ø Merge
1 T. 23 Std.
Gemergte PRs (30 T.)
15

Beschreibung

https://github.com/ElementsProject/elements/blob/cfc10a5dd6c18db140dd1d1e94dfa8cad16bfe36/src/net_processing.cpp#LL136C1-L136C1

```c++
/** Maximum rate of inventory items to send per second.
* Limits the impact of low-fee transaction floods. */
static constexpr unsigned int INVENTORY_BROADCAST_PER_SECOND = 7;
```

Given liquid's higher block rate, this value should probably be higher ([also probably true on Bitcoin itself](https://github.com/bitcoin/bitcoin/pull/27630)) -- perhaps it should be coded as `(4200 / consensusParams.nPowTargetSpacing)` ?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne in src/net_processing.cpp bei der Definition von INVENTORY_BROADCAST_PER_SECOND und verfolge, wie das Limit auf Inventory-Broadcasts angewendet wird. Vergleiche es mit consensusParams.nPowTargetSpacing und der verlinkten Bitcoin-Diskussion; abgeschlossen ist die Arbeit, wenn eine angemessene Rate für Liquids Blockintervall ermittelt und das daraus resultierende Verhalten mit den relevanten bestehenden Tests abgedeckt ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
networking, performance
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
48/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.