From 8c7444576fdcaa317a43694a603aa0ee50ff1798 Mon Sep 17 00:00:00 2001 From: Craig Tiller Date: Fri, 17 Mar 2017 09:57:31 -0700 Subject: [PATCH] Fix tracking of writes: we dont need TrackCounters for bm_fullstack since the fixture does it --- test/cpp/microbenchmarks/bm_fullstack_streaming_ping_pong.cc | 4 ---- 1 file changed, 4 deletions(-) diff --git a/test/cpp/microbenchmarks/bm_fullstack_streaming_ping_pong.cc b/test/cpp/microbenchmarks/bm_fullstack_streaming_ping_pong.cc index 513974fde19..00e37f79121 100644 --- a/test/cpp/microbenchmarks/bm_fullstack_streaming_ping_pong.cc +++ b/test/cpp/microbenchmarks/bm_fullstack_streaming_ping_pong.cc @@ -63,7 +63,6 @@ static void* tag(intptr_t x) { return reinterpret_cast(x); } // the other from server to client): template static void BM_StreamingPingPong(benchmark::State& state) { - TrackCounters track_counters; const int msg_size = state.range(0); const int max_ping_pongs = state.range(1); @@ -152,14 +151,12 @@ static void BM_StreamingPingPong(benchmark::State& state) { fixture->Finish(state); fixture.reset(); state.SetBytesProcessed(msg_size * state.iterations() * max_ping_pongs * 2); - track_counters.Finish(state); } // Repeatedly sends ping pong messages in a single streaming Bidi call in a loop // First parmeter (i.e state.range(0)): Message size (in bytes) to use template static void BM_StreamingPingPongMsgs(benchmark::State& state) { - TrackCounters track_counters; const int msg_size = state.range(0); EchoTestService::AsyncService service; @@ -241,7 +238,6 @@ static void BM_StreamingPingPongMsgs(benchmark::State& state) { fixture->Finish(state); fixture.reset(); state.SetBytesProcessed(msg_size * state.iterations() * 2); - track_counters.Finish(state); } /*******************************************************************************