ElementsProject / ElementsProject/elements

Liquid INV rate probably too low

Aperta
#1,243 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub
enhancement
Lingua principale
C++
Stelle
1.2k
Fork
416
Merge medio
1g 23h
PR unite (30g)
15

Descrizione

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)` ?

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia in src/net_processing.cpp, alla definizione di INVENTORY_BROADCAST_PER_SECOND, e traccia il modo in cui il limite viene applicato agli inventory broadcast. Confrontalo con consensusParams.nPowTargetSpacing e con la discussione di Bitcoin collegata; il lavoro è completato quando viene stabilito un rate appropriato per l’intervallo dei blocchi di Liquid e il comportamento risultante viene coperto dai test esistenti pertinenti.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
networking, performance
Tipo di issue
Bug
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Tranquilla
Chiarezza
Abbastanza chiara
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.