Skip to content

Conversation

@fbac
Copy link
Collaborator

@fbac fbac commented Jun 26, 2025

Some changes are just style changes to make staticcheck linter happy.

@fbac fbac requested a review from a team as a code owner June 26, 2025 16:20
@graphite-app
Copy link

graphite-app bot commented Jun 26, 2025

How to use the Graphite Merge Queue

Add either label to this PR to merge it via the merge queue:

  • Queue - adds this PR to the back of the merge queue
  • Hotfix - for urgent hot fixes, skip the queue and merge this PR next

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

@macroscopeapp
Copy link

macroscopeapp bot commented Jun 26, 2025

Update indexer backfill progress tracking to send UpdateProgress events instead of sleeping when no logs are found

The indexer now sends UpdateProgress events to track backfill progress when no logs are found, replacing the previous sleep mechanism. The changes include:

  • Add UpdateProgressEventSignature(), IsUpdateProgressEvent(), and NewUpdateProgressLog() functions to log_handler.go for handling progress events
  • Modify IndexLogs() function in log_handler.go to check for UpdateProgress events and update the block tracker
  • Update backfill loop in rpc_log_streamer.go to send UpdateProgress events instead of sleeping when no logs are found
  • Rename rpc_streamer package to rpcstreamer and update capitalization conventions across multiple files
  • Add documentation comments following Go conventions

📍Where to Start

Start with the IndexLogs() function in log_handler.go to understand how UpdateProgress events are handled, then review the backfill loop in rpc_log_streamer.go to see how these events are generated.


Macroscope summarized 0bd07fd.

@fbac fbac merged commit 8cbc172 into main Jun 27, 2025
9 checks passed
@fbac fbac deleted the 06-26-indexer_update_progress branch June 27, 2025 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants