Skip to content

sleep instead of queries - #51

Closed
mattac21 wants to merge 1 commit into
mainfrom
ma/no-receipt-off-main
Closed

sleep instead of queries#51
mattac21 wants to merge 1 commit into
mainfrom
ma/no-receipt-off-main

Conversation

@mattac21

@mattac21 mattac21 commented Dec 18, 2025

Copy link
Copy Markdown
Collaborator

Note

Replaces metrics processing with a 15-minute sleep; disables block/receipt queries and stats calculation, returning an empty result.

  • Metrics:
    • Updates ProcessResults in chains/ethereum/metrics/collector.go:
      • Adds 15-minute sleep with before/after logs.
      • Comments out block/receipt fetching and block statistics computation.
      • Returns an empty LoadTestResult instead of computed metrics.

Written by Cursor Bugbot for commit 421581a. This will update automatically on new commits. Configure here.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the final PR Bugbot will review for you during this billing cycle

Your free Bugbot reviews will reset on January 16

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

ByNode: nil, // TODO: we aren't differentiating on node at the moment. not supported.
ByBlock: blockStats,
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Debugging code with 15-minute sleep returns empty results

The ProcessResults function now contains a hardcoded 15-minute time.Sleep and returns an empty LoadTestResult{} with all the actual metrics collection logic commented out. The callers in runner.go expect populated results including TPS, transaction counts, gas statistics, and block data, but they will receive all zero/nil values instead. This appears to be temporary debugging code that completely disables metrics collection functionality.

Fix in Cursor Fix in Web

@mattac21 mattac21 closed this Dec 18, 2025
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.

1 participant