ElementsProject / ElementsProject/elements
Liquid INV rate probably too low
- 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
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