ThreadlocalRandom instead of ThreadLocal
Open
P4
type=other
- Dominant language
- Java
- Stars
- 1.5k
- Forks
- 133
- Avg merge
- 6m
- Merged PRs (30d)
- 7
Description
Hey,
While reading the changes in the newest releases and the code for it, I saw this comment. https://github.com/google/flogger/blob/master/api/src/main/java/com/google/common/flogger/SamplingRateLimiter.java#L55 If I understand the build system correct flogger requires at least java 8, so ThreadLocalRandom(https://docs.oracle.com/javase/8/docs/api/java/util/concurrent/ThreadLocalRandom.html) or `ThreadLocal.withInitial` is available. Is the comment a relict of the past and the code could be changed?
Contributor guide
Assessment
This issue has not been assessed yet.