search

Consensys / teku

Open-source Ethereum consensus client written in Java
https://consensys.io/teku
Apache License 2.0
648 stars 264 forks source link

Upgrade `it.unimi.dsi:fastutil` #8038

Open StefanBratanov opened 4 months ago

StefanBratanov commented 4 months ago

When upgrading it.unimi.dsi:fastutil from 8.5.12 to 8.5.13, there are failing tests in AggregateAttestationValidatorTest. Need to investigate what causes those failings and fix them and only then upgrade.

Captured while working on https://github.com/Consensys/teku/pull/8037

tbenr commented 4 months ago

maybe due to this https://github.com/vigna/fastutil/commit/0f5313322a701e20a4268ec1bf6406addcf696cb ?

tbenr commented 4 months ago

seems like we need to wait 8.5.14, which seems to be on the way

StefanBratanov commented 4 months ago

Good catch. Looks like it.