@zjxxzjwang Please add the following content to your PR description and select a checkbox:

- [ ] `doc` <!-- Your PR contains doc changes -->
- [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
- [ ] `doc-not-needed` <!-- Your PR changes do not impact docs -->
- [ ] `doc-complete` <!-- Docs have been already added -->

计算速率的时间段就是两个任务之间的间隔，如果两个任务之间的时间间隔很小，计算速度就会很大。

很好的观察，@zjxxzjwang看来这个 PR 不仅仅是一个优化，还针对修复无效率统计。

但是，当前的更改会使“主题统计信息”不准确，因为在这种情况下速率不会更新。出于一致性原因，更新速率应以相同的方式在所有代码路径中进行。 一种可能的解决方案是将 PersistentMessageExpiryMonitor 中的 updateRates 方法设为私有，并在返回更新后的速率之前在 getMessageExpiryRate 方法的实现中调用 updateRates。

计算速率的时间段就是两个任务之间的间隔，如果两个任务之间的时间间隔很小，计算速度就会很大。

很好的观察，@zjxxzjwang看来这个 PR 不仅仅是一个优化，还针对修复无效率统计。

但是，当前的更改会使“主题统计信息”不准确，因为在这种情况下速率不会更新。出于一致性原因，更新速率应以相同的方式在所有代码路径中进行。 一种可能的解决方案是将 PersistentMessageExpiryMonitor 中的 updateRates 方法设为私有，并在返回更新后的速率之前在 getMessageExpiryRate 方法的实现中调用 updateRates。

Hello, the rate will continue to be updated ，because the updateRates method of the PersistentTopic class will also be executed at regular intervals, and the time period used to calculate the rate will be fixed (the timing period of the updateRates method). The calculated rate can be explained more strongly. @lhotari

However, the current change would make "topic stats" inaccurate since the rate wouldn't get updated in that case. For consistency reasons, updating the rate should happen in the same way for all code paths.
One possible solution would be to make updateRates method private in PersistentMessageExpiryMonitor and call updateRates in the implementation of getMessageExpiryRate method, before the updated rate is returned.

@zjxxzjwang Please address this review comment. The solution would be somewhat similar as there is in #24067.

但是，当前的更改会使“主题统计信息”不准确，因为在这种情况下速率不会更新。出于一致性原因，更新速率应以相同的方式在所有代码路径中进行。
一种可能的解决方案是将 PersistentMessageExpiryMonitor 中的 updateRates 方法设为私有，并在返回更新后的速率之前在 getMessageExpiryRate 方法的实现中调用 updateRates。

@zjxxzjwang请回复此评论。解决方案与#24067中的有些相似。

Good suggestion, I have made the modifications according to your suggestion. Please help me review it。 @lhotari

@lhotari Hello, please help me review it

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.17%. Comparing base (bbc6224) to head (3fb529b).
Report is 978 commits behind head on master.

@@             Coverage Diff              @@
##             master   #24073      +/-   ##
============================================
+ Coverage     73.57%   74.17%   +0.59%     
+ Complexity    32624    32426     -198     
============================================
  Files          1877     1863      -14     
  Lines        139502   144311    +4809     
  Branches      15299    16457    +1158     
============================================
+ Hits         102638   107041    +4403     
+ Misses        28908    28812      -96     
- Partials       7956     8458     +502