This is an automated email from the ASF dual-hosted git repository. alexey pushed a commit to branch branch-1.18.x in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 12ad004e83ceda874b26a869f8af536555c4a299 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]> (cherry picked from commit eb1c5a110944a0f6d8b34446a8c4e21c2e316b34) Reviewed-on: http://gerrit.cloudera.org:8080/22543 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[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;
