Pregunta 4

Pregunta 4 -> En el commit 94d2027 se cambia m_tx_inventory_to_send de std::set<Wtxid> a std::vector<Wtxid>. ¿Qué se gana y qué se pierde? ¿Cómo se manejan los duplicados ahora?

Los duplicado se manejan cuando se llama a SortMiningScoreWithTopology

(al no tener un set ordenado llamamos a SortMiningScoreWithTopology para tener el vector ordenado)

si, eso intuyo, los duplicado estaran juntos

+1 a @arejula27
en cuanto a duplicados creo que se manejan de dos formas distintas.
Cuando se va a insertar sin pasar por el backlog hay un filtro para ver si estamos añadiendo transacciones que ya conocidas pro el peer.
En el ciclo de “send” hay un condicional para evitar duplicaods en caso que SortMiningScoreWithTopology devuelva el mismo wtxid. Lo cual creo que solo puede pasar si se inserto dos veces pasando por el backlog. Quizas @aj puede confirmar esto ultimo.

hay filtros antes de ese proceso por eso ahi se puede sacrificar eso por tener mas rapidez

Solo añades a la lista cuando aceptas un mensaje en el mempool, que solo se hace una vez, así que no hay duplicados de esa manera; o cuando manualmente “sendrawtransactions” por RPC — en cuyo caso los duplicados son algo culpa tuya. Así que solo intenta evitar enviar los duplicados realmente, en lugar de preocuparte demasiado por tener entradas duplicadas en el backlog.

Claro allí siguen duplicados pero adyacentes

Solo ordenamos parcialmente el backlog, así que solo podemos asegurarnos de que los duplicados en la parte ordenada estarán uno al lado del otro

se eliminan mas adelante cuado estás armando el vInv que es el vector que finalmente se pushea al peer

Tiene sentido. Entiendo que con -privatebroadcast=1 no se generan duplicados usando sendrawtransaction verdad? Juraria que no añade las tx a la mempool.

(esta parte del código es complicada y puede que no sea la mejor forma de hacerlo)

Perfecto

yep, go next 🙂