refactor: increase batch size to get validator indices to 64 (#5743)

ChainSafe · Jul 11, 2023 · 85ff3cf · 85ff3cf
1 parent 7fc999e
commit 85ff3cf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/packages/validator/src/services/indices.ts b/packages/validator/src/services/indices.ts
@@ -7,9 +7,9 @@ import {Metrics} from "../metrics.js";
 
 /**
  * URLs have a limitation on size, adding an unbounded num of pubkeys will break the request.
- * For reasoning on the specific number see: https://github.com/ChainSafe/lodestar/pull/2730#issuecomment-866749083
+ * For reasoning on the specific number see: https://github.com/ethereum/beacon-APIs/pull/328
  */
-const PUBKEYS_PER_REQUEST = 10;
+const PUBKEYS_PER_REQUEST = 64;
 
 // To assist with readability
 type PubkeyHex = string;

diff --git a/packages/validator/src/util/batch.ts b/packages/validator/src/util/batch.ts
@@ -1,6 +1,5 @@
 /**
- * Divide pubkeys into batches, each batch contains at most 5 http requests,
- * each request can work on at most 40 pubkeys.
+ * Convert array of items into array of batched item arrays
  */
 export function batchItems<T>(items: T[], opts: {batchSize: number; maxBatches?: number}): T[][] {
   const batches: T[][] = [];