Bitcoin Core PR Review Club en Español
Reglas Discord ↗

pr-34075

fees: Introduce Mempool Based Fee Estimation to reduce overestimation

Ver Pull Request en GitHub ↗

  1. Introducción
  2. Pregunta 1
    ¿Qué problemas tiene la manera actual en que calculamos las fees con CBlockPolicyEstimator?
  3. Pregunta 2
    En el commit e61ffe0 se limita el uso del struct FeeCalculation y se introduce el struct GetMinimumFeeRateResult. ¿Por qué este cambio? ¿Qué información faltaba o sobraba?
  4. Pregunta 3
    En el commit 579261c se crea FeeSource. ¿Por qué es necesario ese struct y cuál es la diferencia con FeeReason?
  5. Pregunta 4
    ¿Por qué en el commit 04aae1e se evita que el test utilice la mempool?
  6. Pregunta 5
    En el commit d53001e se introduce la clase FeeRateEstimatorManager. ¿Cuál es el papel de esta clase? ¿Por qué esta clase hereda CValidationInterface?
  7. Pregunta 6
    En el commit db99be se introduce el nuevo estimator: MemPoolFeeRateEstimator. ¿Cómo calcula la fee este nuevo estimator? ¿En qué caso CalculatePercentiles no puede calcular la fee? ¿Qué se hace entonces?
  8. Pregunta 7
    En el commit ff51bb2 se añade una caché para el mempoolfeerateestimator. ¿Por qué es importante tener esta caché? ¿Qué condiciones hacen que se vuelva a calcular de nuevo?
  9. Pregunta 8
    En el commit 2303657 se modifica MempoolTransactionsRemovedForBlock en validationinterface para incluir también las transacciones del propio bloque. ¿Por qué el MemPoolFeeRateEstimator necesita las transacciones del bloque y no solo las de la mempool?
  10. Pregunta 9
    ¿Qué está mirando la función IsMempoolHealthy? ¿Qué pasa cuando hay un fork?
  11. Pregunta 10
    En FeeRateEstimatorManager::GetFeeRateEstimate se combina el estimate del CBlockPolicyEstimator con el del MemPoolFeeRateEstimator. ¿En qué condiciones se devuelve cada estimador? ¿Qué pasa si el mempool estimator retorna un error o la mempool no está sana?