add a work pool to fetch check points

[FelixFan1992]

Describe your changes

..

Issue ticket number and link

..

Describe highly relevant files or code snippets that are critical in the review

..

Are there other PRs that should be merged first?

..

[Copilot]

Pull request overview

This PR speeds up chain catch-up by fetching checkpoint data in parallel (while still committing checkpoints in-order) and reduces per-checkpoint DB work in the TransactionsIndexer by caching the decoded transmitter set from the latest ConfigSet event.

Changes:

• Add a concurrent fetch stage to ChainPoller.catchUp, committing checkpoints strictly in sequence order.
• Add a transmitter cache (TTL + “refresh grace” window) to avoid re-querying/re-decoding ConfigSet on every checkpoint.
• Optimize event filtering by precomputing normalized selector type strings; add/adjust unit tests for the new behavior.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
relayer/chainreader/indexer/transactions_indexer.go	Adds transmitter caching and invalidation based on seeing ConfigSet in a checkpoint batch.
relayer/chainreader/indexer/transactions_indexer_unit_test.go	Adds a unit test for detecting ConfigSet presence within a checkpoint batch.
relayer/chainreader/indexer/chain_poller.go	Introduces concurrent checkpoint fetching for catch-up, splits fetch vs commit, and optimizes event filtering comparisons.
relayer/chainreader/indexer/chain_poller_test.go	Makes catch-up tests concurrency-safe (mutex + order-independent assertions).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

relayer/chainreader/indexer/transactions_indexer.go:557

• getTransmitters() documents that callers should not be able to mutate the shared transmitter cache, but on a cache miss it returns the slice from fetchTransmitters() directly. That slice can be mutated by the caller and diverge from the cached copy, and it also violates the function comment.

	return transmitters, nil

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.