Pregunta 8

Pregunta 8 -> ¿Has compilado/testeado el PR? ¿Cómo te organizaste para revisarlo? ¿Hay algo que te sorprendió, edge cases que falten, o algo que te gustaría discutir en grupo?

Como se ha adaptado la PR con cluster mempool? Tambien agrupamos tx en los vectores por paquetes? seria crear ineficiencias, mandarlas en distintos batches al provocar modificaciones en clusters de tx continuamente en tus peers?

en los vectores las transacciones estan desordenadas

Si creo que ya SortMiningScoreWithTopology viene con ese soporte

(Cluster Mempool mejora mucho esto. Antes ordenábamos los TXs según cuántos antepasados en mempool tenían, así que un TX niño que pagaba tarifas altas clasificaba mucho por debajo incluso de uno de cuotas bajas que no tenía padres en mempool)

ahi pensaba que deberia estar, pero no he pododio mirarlo a fondo

no estoy seguro, no he entrado en mucho detalle ahi tbh

Yo asumí que el termino “Topology” era precisamente por esto, por ordenar por paquetes de transacciones padre-hijo

El retraso solo está parcialmente ordenado: solo ordenamos el bit superior que eliminamos. Así que ese bit está ordenado por tarifa y topología. Pero el resto se deja en orden arbitrario.

Bueno yo por mi parte encontré solo 1 nit, lo revisé commit por commit, corrí tests, buildeé y levanté el nodo con esos cambios en signet. Aprendí mucho la verdad

yo estoy revisando aun, tengo un par de comentarios pendientes de publicar en el PR, de momento solo nits 😛

¿A qué llamáis nit?

Cambios minimos que no afectan a la funcionalidad del código

typos, code style..

¿las tx que van en los distintos buckets se duplican? es decir, una misma tx ocupa memoria varias veces?

Las que estan en el backlog si es muy probable

Hoy justo han publicado este post de Boost
https://x.com/Boost_Libraries/status/2047648335754850355

Aunque no recuerdo bien si CtransactionRef resuelve esto? @Pol Espinasa @aj ?

si es un puntero

Bueno entonces corrijo, no es la tx la que se duplica, si no su id

De nada, ¡gracias por acoger!