brpc现在支持返回 Trailers-Only 吗？

[yinchunxiang]

在使用 golang 版本的 grpc 客户端访问 brpc 的 server 的时候，发现不能重试。看 golang 里面代码是在判断 TrailersOnly 的时候失败了。

[wwbmmm]

看代码应该是没有Trailers-Only相关的逻辑

[12epartur3]

看代码应该是没有Trailers-Only相关的逻辑

这个看起来会导致grpc重试功能不生效，咱们brpc这边有计划做一些修改吗

[leaf-potato]

看代码应该是没有Trailers-Only相关的逻辑

想问下，在brpc client场景中server返回的哪些错误码会进行重试呢？我了解的到ELIMIT限流时会进行重试，还有别的情况吗？

[zyearn]

@yinchunxiang Hi, 这看上去grpc stream的代码，目前还不兼容grpc stream.

[zyearn]

@leaf-potato https://github.com/apache/incubator-brpc/blob/master/docs/cn/client.md#%E9%94%99%E8%AF%AF%E5%80%BC%E5%BE%97%E9%87%8D%E8%AF%95

[12epartur3]

看代码应该是没有Trailers-Only相关的逻辑

想问下，在brpc client场景中server返回的哪些错误码会进行重试呢？我了解的到ELIMIT限流时会进行重试，还有别的情况吗？
@leaf-potato
grpc客户端可以配置对应的错误码，遇到对应错误码可以重试
{
"loadBalancingConfig": [
{
"round_robin": {}
}
],
"retryThrottling": {
"maxTokens": 10,
"tokenRatio": 1
},
"methodConfig": [
{
"name": [
{
"service": "my_service,
"method": "my_method"
}
],
"retryPolicy": {
"MaxAttempts": 2,
"InitialBackoff": "0.001s",
"MaxBackoff": "0.001s",
"BackoffMultiplier": 1.0,
"RetryableStatusCodes": [
"UNAVAILABLE",
"RESOURCE_EXHAUSTED"
]
},
"waitForReady": true
}
]
}
参考：
https://github.com/grpc/grpc-go/tree/f51d21267df53c7f4e3533e18fdf5b61c76e005d/examples/features/retry
https://github.com/grpc/proposal/blob/master/A6-client-retries.md#retryable-status-codes

brpc服务器中模拟error，cntl->SetFailed(brpc::ELOGOFF, "test ELOGOFF”);
在grpc的客户端中配置对应错误码 UNAVAILABLE(ELIMIT对应grpc错误码RESOURCE_EXHAUSTED)，发现没有进行重试，后续看到是grpc-go的shouldRetry函数中，判断if !a.s.TrailersOnly()提前退出了。
看grpc的文档貌似应该将错误码返回在TrailersOnly的帧里面，但是brpc返回错误码是带了header。
Most responses are expected to have both headers and trailers but Trailers-Only is permitted for calls that produce an immediate error.
参考：https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md

看了brpc的文档，ELOGOFF和ELIMIT是brpc希望客户端进行重试的两个错误，所以在grpc客户端上配置了对应的错误码UNAVAILABLE和RESOURCE_EXHAUSTED希望实现重试的功能

[12epartur3]

@yinchunxiang Hi, 这看上去grpc stream的代码，目前还不兼容grpc stream.

@zyearn
是brpc的 stream不兼容grpc的stream吗
是否将brpc返回错误码的frame修改为TrailersOnly就可以了？
Most responses are expected to have both headers and trailers but Trailers-Only is permitted for calls that produce an immediate error.
参考：https://github.com/grpc/grpc/blob/master/doc/PROTOCOL-HTTP2.md

[12epartur3]

补充一下问题复现：

[yinchunxiang]

@yinchunxiang Hi, 这看上去grpc stream的代码，目前还不兼容grpc stream.

可以通过在 brpc 服务端继承 RetryPolicy来实现支持 TrailersOnly，从而达到让 grpc 重试吗？ @zyearn