Details
-
Sub-task
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
None
-
None
-
None
-
Reviewed
Description
While looking over Findbugs/Spotbugs findings on HBASE-26555 I noticed we have some violations of Spotbugs rules regarding lack of Random object re-use, and generally have a lot of static Random and SecureRandom objects in our code, especially in tests, where ThreadLocalRandom would be more than sufficient.
This is nitty stuff but let's clean it up. It will reduce noise on the analysis reports and SecureRandom is unnecessarily slow when used wherever we don't need it.
Attachments
Issue Links
- links to