Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

core/txpool: add some txpool metrics #505

Open
wants to merge 8 commits into
base: pre-release
Choose a base branch
from

Conversation

c98tristan
Copy link
Contributor

This pull request includes several changes to replace the use of counters with meters for transaction metrics in the lending pool, order pool, and transaction pool. The most important changes include modifying various methods to use meters instead of counters and updating the metric definitions.

Metrics Update:

  • core/tx_pool.go: Replaced counters with meters for various transaction metrics, including pendingDiscardMeter, pendingReplaceMeter, queuedDiscardMeter, and invalidTxMeter. Added new gauges for pendingGauge, queuedGauge, and localGauge. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]
# TYPE txpool_invalid gauge
txpool_invalid 19

# TYPE txpool_local gauge
txpool_local 0

# TYPE txpool_pending gauge
txpool_pending 6

# TYPE txpool_pending_discard gauge
txpool_pending_discard 0

# TYPE txpool_pending_nofunds gauge
txpool_pending_nofunds 0

# TYPE txpool_pending_ratelimit gauge
txpool_pending_ratelimit 0

# TYPE txpool_pending_replace gauge
txpool_pending_replace 0

# TYPE txpool_queued gauge
txpool_queued 0

# TYPE txpool_queued_discard gauge
txpool_queued_discard 0

# TYPE txpool_queued_nofunds gauge
txpool_queued_nofunds 0

# TYPE txpool_queued_ratelimit gauge
txpool_queued_ratelimit 0

# TYPE txpool_queued_replace gauge
txpool_queued_replace 0

# TYPE txpool_underpriced gauge
txpool_underpriced 0

# TYPE txpool_valid gauge
txpool_valid 7763

Reference:

@c98tristan c98tristan marked this pull request as draft February 14, 2025 11:29
@c98tristan c98tristan marked this pull request as ready for review February 27, 2025 10:33
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