makabakaboom commented on pull request #3191: URL: https://github.com/apache/rocketmq/pull/3191#issuecomment-890651759
> > csListMinute，csListHour，csListDay列表分别保存一分钟、一小时、一天的点，分别对应有6、6、24个点，computeStatsData通过调用栈我看用途大多数都是去计算系统中一些统计值，如topic一段时间内的消息量、TPS等统计值。按照原实现，比如我看到的topic的小时级别数据和天级别的数据就具有一定的延迟性，比如小时级别的数据我就需要10分钟后才能精确，天的数据就有1小时延迟。修改代码只是在统计分钟级别的时候去更新了小时和天级别的最后一个值的数据，这样小时级别和天级别还是原设计的6、24个点，只是小时改为统计1h+10min内的实时值，一天改为统计1D + 1h内的实时值。或许这里修改不是最优的实现方式，但是我认为这里确实存在统计延迟 > > 你是对的,我的问题,没看仔细.有个问题问下,为啥不放在synchronized代码块里面了 考虑synchronized代码块里面主要是对 对应列表有影响的放在里面，且每个采样方法里面都有对应列表锁，而一小时和一天修改跟一分钟列表无关系，没必要在一分钟的synchronized代码块里面 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
