Marton Greber has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/22520 )

Change subject: [rpc-test] fix missing field in 
TimedOutOnResponseMetricServiceQueue
......................................................................

[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]>
---
M src/kudu/rpc/rpc-test.cc
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Marton Greber: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/22520
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I097916c6ffbb7b7368d67233b47427db2cefd185
Gerrit-Change-Number: 22520
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>

Reply via email to