squbs icon indicating copy to clipboard operation
squbs copied to clipboard

Max value for incoming http connections should be captured via `MaterializationMetricsCollector`

Open anilgursel opened this issue 7 years ago • 7 comments

anilgursel avatar Jul 31 '18 17:07 anilgursel

Currently, we see active, created, terminated rates. But, having a max would be useful as well, which we had in the past.

anilgursel avatar Jul 31 '18 17:07 anilgursel

I'm looking into this issue. We can implement it in the MaterializationMetricsCollector. But IMO this should not be an issue for MaterializationMetricsCollector but rather in the JmxReporter.

akara avatar Aug 02 '18 17:08 akara

The active, created and terminated are three separate metrics and independent from JmxReporter's view. Not sure, how can that be achieved that way.

anilgursel avatar Aug 06 '18 15:08 anilgursel

These three are totally independent counters. created and terminated only get incremented. Max is always the current value. active gets incremented and decremented. This is where the max makes sense. But considering the concurrency on this metrics, what is even a reliable way to obtain a max short of putting another lock around both increment and max?

akara avatar Aug 06 '18 16:08 akara

It is best effort. It can be another gauge or counter that shows the max of active ever reached.

anilgursel avatar Aug 06 '18 17:08 anilgursel

That's exactly what I mean, should derivation of the max be a separate counter, or should it be the monitoring infrastructure?

akara avatar Aug 06 '18 17:08 akara

If Jmx Reporter already has the capability, I think we can utilize that, or potentially use a different metric that also keeps track of the max.

anilgursel avatar Aug 06 '18 17:08 anilgursel