This is an automated email from the ASF dual-hosted git repository.
mgreber pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
The following commit(s) were added to refs/heads/master by this push:
new eb1c5a110 [rpc-test] fix missing field in
TimedOutOnResponseMetricServiceQueue
eb1c5a110 is described below
commit eb1c5a110944a0f6d8b34446a8c4e21c2e316b34
Author: Alexey Serbin <[email protected]>
AuthorDate: Fri Feb 21 19:14:57 2025 -0800
[rpc-test] fix missing field in TimedOutOnResponseMetricServiceQueue
With protobuf 3.21.9, the TimedOutOnResponseMetricServiceQueue scenario
of the TestRpc suite started reporting an error like below, that
sometimes leads to failures as reported by [1]. This patch addresses
the issue.
[libprotobuf ERROR
thirdparty/src/protobuf-3.21.9/src/google/protobuf/message_lite.cc:133] Can't
parse message of type "kudu.rpc_test.SleepRequestPB" because it is missing
required fields: sleep_micros
W20250221 18:47:46.831769 207179 service_if.cc:72] invalid parameter for
call kudu.rpc_test.CalculatorService.Sleep: missing fields: sleep_micros
This is a follow-up to 0d837c5aac482fbe46170e26ac45e5ad951eecec.
[1]
http://dist-test.cloudera.org:8080/diagnose?key=cc8af164-eda5-11ef-a520-42010af00016
Change-Id: I097916c6ffbb7b7368d67233b47427db2cefd185
Reviewed-on: http://gerrit.cloudera.org:8080/22520
Tested-by: Kudu Jenkins
Reviewed-by: Marton Greber <[email protected]>
---
src/kudu/rpc/rpc-test.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/kudu/rpc/rpc-test.cc b/src/kudu/rpc/rpc-test.cc
index ba5b04623..c8e5b01e9 100644
--- a/src/kudu/rpc/rpc-test.cc
+++ b/src/kudu/rpc/rpc-test.cc
@@ -1469,6 +1469,7 @@ TEST_P(TestRpc, TimedOutOnResponseMetricServiceQueue) {
// processed by the only thread in the RPC service thread pool. Eventually,
// it should time out.
SleepRequestPB req1;
+ req1.set_sleep_micros(0);
req1.set_return_app_error(true);
SleepResponsePB resp1;
RpcController ctl1;
