Skip to content

Conversation

@li4wang
Copy link
Contributor

@li4wang li4wang commented Oct 16, 2025

This PR contains the following changes for supporting client-triggered op count metrics for more visibility of client triggered load and pattern.

  • ServerMetrics.java: Added counter definitions and initialization
  • FinalRequestProcessor.java: Implemented operation counting in the main request processing pipeline
  • ServerMetricsOpCountTest.java: Added unit tests

@li4wang li4wang force-pushed the ZOOKEEPER-4983 branch 4 times, most recently from 2b44c44 to 436ef46 Compare October 21, 2025 02:31
@li4wang li4wang force-pushed the ZOOKEEPER-4983 branch 4 times, most recently from 16b0d7b to b37eefd Compare December 13, 2025 03:56
@li4wang
Copy link
Contributor Author

li4wang commented Dec 13, 2025

@eolivelli @kezhuw @anmolnar Would you mind reviewing the PR? Thanks

/**
* Operation count metrics
*/
TOTAL_OP_COUNT = metricsContext.getCounter("total_op_count");
Copy link
Contributor

Choose a reason for hiding this comment

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

what about adding a common prefix to all these metrics ?

like "frp_" (final request processor) or "final_request_processor_"

This way the metrics are naturally grouped (and when you explore them with Grafana or build a dashboard your life is much easier)

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.

2 participants