From a4942a012df21d0feb73c4f98f48e248b33a4a51 Mon Sep 17 00:00:00 2001
From: vjpai <vpai@google.com>
Date: Mon, 21 Mar 2016 12:44:02 -0700
Subject: [PATCH] @jtattermusch correctly pointed out that we are not setting
payload config in our server config. This affect any generic server tests
that use anything other than 0-byte responses: essentially, server-streaming
or bidi throughput tests.
---
test/cpp/qps/generic_async_streaming_ping_pong_test.cc | 1 +
test/cpp/qps/qps_driver.cc | 4 ++++
2 files changed, 5 insertions(+)
diff --git a/test/cpp/qps/generic_async_streaming_ping_pong_test.cc b/test/cpp/qps/generic_async_streaming_ping_pong_test.cc
index 77ed11f287..fc06cddfef 100644
--- a/test/cpp/qps/generic_async_streaming_ping_pong_test.cc
+++ b/test/cpp/qps/generic_async_streaming_ping_pong_test.cc
@@ -62,6 +62,7 @@ static void RunGenericAsyncStreamingPingPong() {
ServerConfig server_config;
server_config.set_server_type(ASYNC_GENERIC_SERVER);
server_config.set_async_server_threads(1);
+ *server_config.mutable_payload_config() = client_config.payload_config();
const auto result =
RunScenario(client_config, 1, server_config, 1, WARMUP, BENCHMARK, -2);
diff --git a/test/cpp/qps/qps_driver.cc b/test/cpp/qps/qps_driver.cc
index 69fb4d75e8..f9bd01b2a1 100644
--- a/test/cpp/qps/qps_driver.cc
+++ b/test/cpp/qps/qps_driver.cc
@@ -171,6 +171,10 @@ static void QpsDriver() {
server_config.set_core_limit(FLAGS_server_core_limit);
}
+ if (FLAGS_bbuf_resp_size >= 0) {
+ *server_config.mutable_payload_config() = client_config.payload_config();
+ }
+
if (FLAGS_secure_test) {
// Set up security params
SecurityParams security;
--
GitLab