Attention is currently required from: neels.
iedemam has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-bsc/+/27081 )

Change subject: stats: new trackers for lchan life duration
......................................................................


Patch Set 18:

(1 comment)

Patchset:

PS18:
Hi Neels,

I appreciate the review. I currently have reduced availability due to some 
vacation days but will address the comments and update the patchset early next 
week.

One clarification before I address the comments: I definitely want to record 
milliseconds (if only for clarity...no one uses decaseconds) but don't need to 
update the timer each millisecond. Is it possible to keep resolution at 
milliseconds but only trigger updates every 1/10th of a second? This would be 
ideal to me.

Everything else looks good to me. I just need a chance to implement + post back 
here.

Thanks again,
-Michael



--
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/27081
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I1b0670c47cb5e0b7776eda89d1e71545ba0e3347
Gerrit-Change-Number: 27081
Gerrit-PatchSet: 18
Gerrit-Owner: iedemam <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: neels <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: neels <[email protected]>
Gerrit-Comment-Date: Fri, 01 Apr 2022 13:31:57 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to