After PR #3554 we use the new Logs API from commonware, but we still collect all dealer logs at once and call finalize/observe immediately — missing the incremental verification benefit.
We should call Logs::record() as each dealer log is finalized in a block, and Logs::pre_verify() periodically (e.g. after each batch of new logs), so that signature verification is amortized over the epoch rather than done all at once at the boundary block.
This matters more as the validator set grows.
After PR #3554 we use the new
LogsAPI from commonware, but we still collect all dealer logs at once and callfinalize/observeimmediately — missing the incremental verification benefit.We should call
Logs::record()as each dealer log is finalized in a block, andLogs::pre_verify()periodically (e.g. after each batch of new logs), so that signature verification is amortized over the epoch rather than done all at once at the boundary block.This matters more as the validator set grows.