fix: partially downloaded batch

ChainSafe · Jan 13, 2025 · 765bd6f · 765bd6f
1 parent e8bc729
commit 765bd6f
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/packages/beacon-node/src/sync/range/chain.ts b/packages/beacon-node/src/sync/range/chain.ts
@@ -436,11 +436,13 @@ export class SyncChain {
           if (hasPostDenebBlocks) {
             Object.assign(downloadInfo, {blobs, dataColumns});
           }
-          this.logger.debug("Downloaded batch", {id: this.logId, ...batch.getMetadata(), ...downloadInfo, peer});
+          this.logger.debug("Fully downloaded batch", {id: this.logId, ...batch.getMetadata(), ...downloadInfo, peer});
+          this.triggerBatchProcessor();
         } else {
-          this.logger.debug("Partially downloaded batch", {id: this.logId, ...batch.getMetadata(), peer});
+          const pendingDataColumns = res.result.pendingDataColumns?.join(",");
+          this.logger.debug("Partially downloaded batch", {id: this.logId, ...batch.getMetadata(), pendingDataColumns, peer});
+          // the flow will continue to call triggerBatchDownloader() below
         }
-        this.triggerBatchProcessor();
       } else {
         this.logger.verbose("Batch download error", {id: this.logId, ...batch.getMetadata()}, res.err);
         batch.downloadingError(); // Throws after MAX_DOWNLOAD_ATTEMPTS