Alexey Serbin has posted comments on this change. ( http://gerrit.cloudera.org:8080/15197 )
Change subject: KUDU-2996: trace when rpc_duration_too_long_ms exceeded ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/15197/1/src/kudu/rpc/rpcz_store.cc File src/kudu/rpc/rpcz_store.cc: http://gerrit.cloudera.org:8080/#/c/15197/1/src/kudu/rpc/rpcz_store.cc@270 PS1, Line 270: << "Request Metrics: " << call->trace()->MetricsAsJSON() << "\n"; If we are dumping both time deltas and metrics below, why do we need to duplicate metrics here? Maybe, remove this then or set INCLUDE_TIME_DELTAS for the DumpToString() below? -- To view, visit http://gerrit.cloudera.org:8080/15197 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I20d9b5e318f4a987051f1f7c2cd5b1d92c61bd05 Gerrit-Change-Number: 15197 Gerrit-PatchSet: 1 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 10 Feb 2020 18:54:52 +0000 Gerrit-HasComments: Yes
