Aaryamann Challani
Aaryamann Challani
@xgreenx I am going to revert your change, this PR is only meant for master integration
synced a testnet node and had 2 local nodes sync from it at the same time -
> LGTM. I have a side question of whether the cache should be cleared in case of a DB rollback, to avoid inconsistencies? that's a good question! i wonder if...
> I think the usage of the concurrent LRU cache would solve all race conditions between updating cache and actually usage of it. Plus it will remove the usage of...
assigning to myself
@kaiserd should we update the acceptance criteria here to just include the raw rfc for the current implementation? imo we shouldn't be including the padding in it yet since there...
Updated issue description to include https://github.com/vacp2p/rfc/issues/560
Reopening this since github auto-closed it ~ we still have to address the new editor issue
hmm, i think it should return a value only if the bundle is "wholly" included in the range.
yes please, all the `fault-proving` feature flags