wwbmmm commented on code in PR #2273:
URL: https://github.com/apache/brpc/pull/2273#discussion_r1227486531
##########
src/brpc/channel.h:
##########
@@ -119,6 +120,17 @@ struct ChannelOptions {
// Default: NULL
const RetryPolicy* retry_policy;
+ // Backoff request before every retry. The interface
+ // is defined in src/brpc/retry_backoff.h
+ // This object is NOT owned by channel and should
+ // remain valid when channel is used.
+ // Default: NULL
+ const RetryBackoffPolicy* retry_backoff_policy;
+ // Enable retry backoff in pthread.
+ // Note it will block the pthread.
+ // Default: false
+ bool enable_retry_backoff_in_pthread;
Review Comment:
这个参数,是否可以给RetryBackoffPolicy加一个虚方法来返回?
这样就不用加太多的参数在ChannelOptions里
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]