From 8c7444576fdcaa317a43694a603aa0ee50ff1798 Mon Sep 17 00:00:00 2001
From: Craig Tiller <ctiller@google.com>
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 513974fde1..00e37f7912 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<void*>(x); }
// the other from server to client):
template <class Fixture, class ClientContextMutator, class ServerContextMutator>
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 <class Fixture, class ClientContextMutator, class ServerContextMutator>
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);
}
/*******************************************************************************
--
GitLab