Bhanu Pulluri
Bhanu Pulluri
Hi @siladu , thanks for pointing it out. I believe @matkt meant that the change doesn't fix the bug but doesn't introduce a new bug . I'll check on what...
> This also happens in other circumstances, notably when running in docker. > > It's the "Correct Way To Do It" that `besu` doesn't use the `root` user within the...
- Added a Frequent multicaster class to resend a message to validators at regular intervals . This is used for round change messages upon using fast-recovery option - Moved early-round-change...
@matthew1001 is working on moving QBFT functionality to a plugin . This would be a part of that plugin then.
Fixed the compile error and merged latest main
> @pullurib from the commits it looked like maybe you tried an approach using RoundTimer first...is there a reason that didn't work out and you settled on a separate retrying...
@macfarla Sorry lost track of this . I spent time looking into it but adapting fastsync to snapsync is not a trivial change to include in this PR . I...
> @pullurib I haven't had a detailed look at this yet. Will the non-validator nodes validate QBFT messages before gossiping them? I am concerned that if the messages aren't validated...
> few comments from a (very) brief review. Refactoring the tests to use QBFT could be a separate PR, non-breaking. Let's be sure we aren't introducing any instability in ATs...