chewbranca commented on code in PR #5213:
URL: https://github.com/apache/couchdb/pull/5213#discussion_r1807069156
##########
src/fabric/priv/stats_descriptions.cfg:
##########
@@ -26,3 +26,53 @@
{type, counter},
{desc, <<"number of write quorum errors">>}
]}.
+
+
+%% fabric_rpc worker stats
+%% TODO: decide on which naming scheme:
+%% {[fabric_rpc, get_all_security, spawned], [
+%% {[fabric_rpc, spawned, get_all_security], [
+{[fabric_rpc, get_all_security, spawned], [
Review Comment:
Another note, `spawned` is ubiquitous as these processes are all being
spawned in the same manner that we track directly, but the `processed` vs
`returned` vs `etc` is far more operation specific and we should look at what
other types of data are important to track here, for instance, what would be
pertinent data to track out of Clouseau/Nouveau requests? Similarly, even
singular doc API operations might have relevant magnitudes, eg we should
probably track the number of doc revisions the worker had to load to complete
the request.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
