search

mosn / holmes

self-aware Golang profile dumper
Apache License 2.0
1.06k stars 135 forks source link

Update zh.md #108

Closed LvBay closed 2 years ago

LvBay commented 2 years ago

修正中文描述

mosn-community-bot[bot] commented 2 years ago

Hi @LvBay, welcome to mosn community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

Jun10ng commented 2 years ago

hi LvBay

collectInterval参数会被初始化为一个 ticker，然后在for循环内捕获ticker的信号，从而定期探测应用的各项性能指标。

文档内描述"建议CollectInterval要大于1s“是因为，for循环内部的逻辑耗时已经大于1s了，如果设置为小于1s则没有意义。

具体的代码逻辑可以看下这里。

文档上是我表述得不够清楚，欢迎提个PR补充一下文档说明(zh/en) 和注释，以防其他同学有相同的疑问。

LvBay commented 2 years ago

@Jun10ng 你好，文档应该是想表达“生产环境该值建议设置为大于10s”吧，我看文档文档写成了“1s”

Jun10ng commented 2 years ago

@Jun10ng 你好，文档应该是想表达“生产环境该值建议设置为大于10s”吧，我看文档文档写成了“1s”

没有写错，最小的hCollectInterval只能支持到1s左右，因为探测工作都要执行1s左右了。