Bitshala / BitcoinCore-PR-Review-Club

Bitcoin Core PR Review Organising repo
10 stars 2 forks source link

Review club : p2p - Fill reconciliation sets (Code review - part 2) #61

Open Prabhat1308 opened 1 month ago

Prabhat1308 commented 1 month ago

Session Details

Date : 13-06-2024 Time: IST 20:00 (UTC 14:30) Link : p2p: Fill reconciliation sets (Erlay) BIP : 330 Difficulty : High

Notes

This PR introduces tracking of per-peer reconciliation sets containing transactions to be exchanged efficiently using minisketch. Currently all the transaction are flooded to each peer via flooding algorithm .

The section to be reviewed is txreconciliation.h/txreconciliation.cpp and txreconciliation_tests.cpp.

Questions

  1. Why testing with Boost library and not use python tests ?
  2. Functional tests vs Unit Tests .
  3. MAX_SET_SIZE for wtixd is capped at 3000 . Why is that ?

Learnings

sketches erlay multi-threaded architecture

Pre-Requisites

PR review club - BIP330 guide to run tests in bitcoin-core