uber-archive/cherami-server

ConcurrentMap size calculation is wrong?

Opened this issue · 0 comments

wxing1292 commented

Please see the followings:

  1. Remove decrement the counter with no checking of whether key exists or not:
    https://github.com/uber/cherami-server/blob/master/common/concurrentmap.go#L173

  2. Put increase the counter with no checking of whether key exists or not:
    https://github.com/uber/cherami-server/blob/master/common/concurrentmap.go#L148