apache / apache/iotdb

Save and re-use this "Random".

未关闭
#5,880 2 条评论 0 个 reaction 已指派 1 人 已被 @azhsmesos 认领 在 GitHub 查看
Easy-Fixed
主要语言
Java
星标
6.4k
派生
1.2k
平均合并
1 天 23 小时
30 天内合并 PR
115

描述

Code Path: confignode/src/main/java/org/apache/iotdb/confignode/manager/PartitionManager.java
image
Creating a new Random object each time a random value is needed is inefficient and may produce numbers which are not random depending on the JDK. For better efficiency and randomness, create a single Random, then store, and reuse it.

The Random() constructor tries to set the seed with a distinct value every time. However there is no guarantee that the seed will be random or even uniformly distributed. Some JDK will use the current time as seed, which makes the generated numbers not random at all.

This rule finds cases where a new Random is created each time a method is invoked.

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。