Bright Chen

Results 217 comments of Bright Chen

> 嗯嗯,理解这个场景了。其实跟RetryPolicy一样,让用户决定是否要继续发请求。 > > 或许backup_request_policy可以都支持: > > 1. 支持动态timeout; > 2. 支持在HandleBackupRequest拦截。 @icexin @GreateCode @xiaoma2015 @NuttyNull 提了个[PR](https://github.com/apache/brpc/pull/2734),大家看看有什么建议?

308是server响应的状态码。如果server响应308状态码的话,http rpc就会失败。这块逻辑没有改动过。

> 改成clock_gettime对性能是否有损?cpuwide_time的使用频率还挺高的 @wwbmmm 好像在百度内部才用rdtsc指令吧,开源用的是clock_gettime。 https://github.com/apache/brpc/blob/d5cdbe69a653684f6593b3448a685a27eb2621a1/src/butil/time.h#L274-L285

> 那这个问题是不是可以close了? 是的。 > 可以参考 [dpdk](https://github.com/DPDK/dpdk/blob/a86f381b826660e88794754c41d3aec79ce9b87c/lib/eal/x86/rte_cycles.c#L107) 的做法读 cpuid。 使用rdtsc来提升cpuwide_time_ns性能倒是个方向。

examples也不全,可以考虑补全一下。

[https://github.com/apache/brpc/blob/c24e641009fa0305f0efebce694d0dad886c85ca/test/brpc\_http\_rpc\_protocol\_unittest.cpp#L1131](https://github.com/apache/brpc/blob/c24e641009fa0305f0efebce694d0dad886c85ca/test/brpc_http_rpc_protocol_unittest.cpp#L1131) 是不是改成`CHECK(test_header != NULL)`就可以兼容新旧版本了?

有设置FLAGS_usercode_in_pthread这个gflag吗?

有用BTHREAD_ATTR_PTHREAD启动bthread吗? 默认情况下,CallMethod是跑在bthread上的,不会阻塞worker pthread。