Skip to content

Commit 8934804

Browse files
authored
chore(prover-client): demote empty addTxs log from warn to verbose (#24593)
`CheckpointSubTreeOrchestrator.addTxs` logs `Provided no txs to addTxs.` at **warn** level, but this path is expected control flow, not an anomaly: `checkpoint-prover.ts` calls `this.subTree.addTxs(processed)` unconditionally for every block in the checkpoint, so every empty block hits this early return (the empty block is then finalized via `setBlockCompleted`). On low-traffic networks this fires constantly — ~100 times in 4h on staging-internal — polluting warn-level log reviews with noise. Demote it to `verbose`. No behavior change; no tests assert on this message. Replaces #24591 and #24592 (closed), per request to land this as a single PR via the spartan-v5 merge train. Spotted during a staging-internal log review requested by @spalladino in Slack. --- *Created by [claudebox](https://claudebox.work/v2/sessions/ac050bae673edac4) · group: `slackbot`*
1 parent 42eea48 commit 8934804

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

yarn-project/prover-client/src/orchestrator/checkpoint-sub-tree-orchestrator.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -323,7 +323,7 @@ export class CheckpointSubTreeOrchestrator extends ProvingScheduler {
323323
if (!txs.length) {
324324
// Empty block: setBlockCompleted handles this without addTxs being called. Bail to
325325
// avoid the throw below (we cannot find the blockNumber without any txs).
326-
this.logger.warn(`Provided no txs to addTxs.`);
326+
this.logger.verbose(`Provided no txs to addTxs.`);
327327
return;
328328
}
329329

0 commit comments

Comments
 (0)