Tuvie commented on code in PR #1836:
URL: https://github.com/apache/incubator-brpc/pull/1836#discussion_r939473930
##########
src/brpc/server.cpp:
##########
@@ -701,6 +703,28 @@ static bool CreateConcurrencyLimiter(const
AdaptiveMaxConcurrency& amc,
return true;
}
+#if BRPC_WITH_RDMA
+static bool OptionsAvailableOverRdma(const ServerOptions* opt) {
+ if (opt->rtmp_service) {
+ LOG(WARNING) << "RTMP is not supported by RDMA";
+ return false;
+ }
+ if (opt->has_ssl_options()) {
+ LOG(WARNING) << "SSL is not supported by RDMA";
+ return false;
+ }
+ if (opt->nshead_service) {
Review Comment:
可以支持，之前的版本也允许。但是事实上不建议这么做，因为http头本身也很重。所以这个版本给取消了
--
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]
