jitsi / jicofo

JItsi COnference FOcus is a server side focus component used in Jitsi Meet conferences.
Apache License 2.0
316 stars 347 forks source link

Limit how frequently visitorCount is updated on member leave. #1099

Closed JonathanLennox closed 1 year ago

codecov[bot] commented 1 year ago

Codecov Report

Merging #1099 (3e5ac08) into master (9a6558b) will decrease coverage by 0.02%. The diff coverage is 35.71%.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/jitsi/jicofo/pull/1099/graphs/tree.svg?width=650&height=150&src=pr&token=wdwr26KPKY&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi)](https://app.codecov.io/gh/jitsi/jicofo/pull/1099?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) ```diff @@ Coverage Diff @@ ## master #1099 +/- ## ============================================ - Coverage 27.94% 27.93% -0.02% - Complexity 500 501 +1 ============================================ Files 129 129 Lines 7678 7678 Branches 1076 1077 +1 ============================================ - Hits 2146 2145 -1 + Misses 5265 5264 -1 - Partials 267 269 +2 ``` | [Impacted Files](https://app.codecov.io/gh/jitsi/jicofo/pull/1099?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) | Coverage Δ | | |---|---|---| | [...in/kotlin/org/jitsi/jicofo/util/RateLimitedStat.kt](https://app.codecov.io/gh/jitsi/jicofo/pull/1099?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi#diff-amljb2ZvL3NyYy9tYWluL2tvdGxpbi9vcmcvaml0c2kvamljb2ZvL3V0aWwvUmF0ZUxpbWl0ZWRTdGF0Lmt0) | `76.00% <27.27%> (-1.56%)` | :arrow_down: | | [...tsi/jicofo/conference/JitsiMeetConferenceImpl.java](https://app.codecov.io/gh/jitsi/jicofo/pull/1099?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi#diff-amljb2ZvL3NyYy9tYWluL2phdmEvb3JnL2ppdHNpL2ppY29mby9jb25mZXJlbmNlL0ppdHNpTWVldENvbmZlcmVuY2VJbXBsLmphdmE=) | `44.93% <66.66%> (-0.07%)` | :arrow_down: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/jitsi/jicofo/pull/1099/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) ------ [Continue to review full report in Codecov by Sentry](https://app.codecov.io/gh/jitsi/jicofo/pull/1099?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi) > `Δ = absolute (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://app.codecov.io/gh/jitsi/jicofo/pull/1099?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). Last update [9a6558b...3e5ac08](https://app.codecov.io/gh/jitsi/jicofo/pull/1099?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=jitsi).