nickva commented on PR #6068: URL: https://github.com/apache/couchdb/pull/6068#issuecomment-4992780757
> > Wonder if we can just insert it in two places: in dec_count and inc_count. Every time we bump the internal count we also update metric? > > I was hesitant about coupling it to the stats system, but it definitely looks much cleaner this way. PR updated 👍 They are related and it makes sense to keep both closer to each other. In the future if we add another place we change the process count it won't forget to bump the metric. -- 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]
