Merge pull request #1887 from ethereum-optimism/fix/dtl-large-query

dtl: smaller range for filter query

.changeset/clean-jars-check.md

+---
+'@eth-optimism/data-transport-layer': patch
+---
+
+Smaller filter query for searching for L1 start height. This number should be configured so that the search does not need to happen because using a smaller filter will cause it to take too long.

packages/data-transport-layer/src/services/l1-ingestion/service.ts

@@ -167,7 +167,7 @@ export class L1IngestionService extends BaseService<L1IngestionServiceOptions> {
         startingL1BlockNumber = this.options.l1StartHeight
       } else {
         this.logger.info(
-          'Attempting to find an appropriate L1 block height to begin sync...'
+          'Attempting to find an appropriate L1 block height to begin sync. This may take a long time.'
         )
         startingL1BlockNumber = await this._findStartingL1BlockNumber()
       }
@@ -453,12 +453,18 @@ export class L1IngestionService extends BaseService<L1IngestionServiceOptions> {
 
   private async _findStartingL1BlockNumber(): Promise<number> {
     const currentL1Block = await this.state.l1RpcProvider.getBlockNumber()
+    const filter =
+      this.state.contracts.Lib_AddressManager.filters.OwnershipTransferred()
+
+    for (let i = 0; i < currentL1Block; i += 2000) {
+      const start = i
+      const end = Math.min(i + 2000, currentL1Block)
+      this.logger.info(`Searching for ${filter} from ${start} to ${end}`)
 
-    for (let i = 0; i < currentL1Block; i += 1000000) {
       const events = await this.state.contracts.Lib_AddressManager.queryFilter(
-        this.state.contracts.Lib_AddressManager.filters.OwnershipTransferred(),
-        i,
-        Math.min(i + 1000000, currentL1Block)
+        filter,
+        start,
+        end
       )
 
       if (events.length > 0) {