-
Notifications
You must be signed in to change notification settings - Fork 797
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
retrystate默认设置处理问题 #2330
Comments
这么改的话,会不会有就默认值的问题呢? |
我觉得就是正常场景,这个重试的策略就应该由用户自己去定义,而不应该给默认给重试三次。由用户选择性的做,而不应该强制做。 |
这里需要考虑向后兼容呢,可能有些老用户,升级了就不重试了,这个影响就大了呢 需求我觉得是合理的,不过得想个能向后兼容的实现 |
可以设置成,一旦有用户配置的重试次数,那么就以用户的配置为准? |
我觉得可以,retryPolicy不为nil则该值设置的重试次数,反之使用默认设置的3次
罗泽轩 ***@***.***> 于 2023年6月12日周一 20:36写道:
… 可以设置成,一旦有用户配置的重试次数,那么就以用户的配置为准?
—
Reply to this email directly, view it on GitHub
<#2330 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ACBO37TT3IN6ESQN4JVHRADXK4EOBANCNFSM6AAAAAAZCY6XT4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
感觉新增配置项,理解成本会增高 |
这里有问题呢,用户原来配置 RetryPolicyConfig,但是没有配置 num_retries,原来的逻辑是默认三次呢 如果没有历史负担,我是倾向于直接改的,但是有历史负担,还是要考虑下权衡,其实也是不太好权衡的 |
我觉得这种可能性小,他既然到知道用配置了,肯定是知道重试机制的,不存在不配置次数吧,这个配置一共也没几个配置项 |
这里不能假设肯定会配置呢,没有强制要求配置的,只能说可能性相对小 但是吧,重试这种东西,可大可小,而且有问题了,也不太好发现,真出了问题,其实是比较坑的 如果觉得加配置比较麻烦的话,可以新增一个全局变量当配置,通过变量来控制行为变化,然后暴露一个全局 API 函数来修改这个配置 |
https://github.com/mosn/mosn/blob/f20464f60fe98cc69684bf3d510ce88214227841/pkg/proxy/retrystate.go#LL50C34-L50C35
这里感觉可以直接改成如下
由用户自定义即可
The text was updated successfully, but these errors were encountered: