Skip to content

Commit

Permalink
Revert "config: optimize RepeatedPtrUtil::debugString at non-debug lo…
Browse files Browse the repository at this point in the history
…g levels. (#2984)" (#3054)

This reverts commit 4874315.

As pointed out by @alyssawilk, this doesn't make sense given #2751. I
think the profile that motivated this must have been created before
the Google import included #2751.

Signed-off-by: Harvey Tuch <htuch@google.com>
  • Loading branch information
htuch authored and mattklein123 committed Apr 11, 2018
1 parent 1316466 commit 4843fcc
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions source/common/config/grpc_mux_subscription_impl.h
Original file line number Diff line number Diff line change
Expand Up @@ -55,10 +55,8 @@ class GrpcMuxSubscriptionImpl : public Subscription<ResourceType>,
stats_.update_attempt_.inc();
version_info_ = version_info;
stats_.version_.set(HashUtil::xxHash64(version_info_));
if (ENVOY_LOG_CHECK_LEVEL(debug)) {
ENVOY_LOG(debug, "gRPC config for {} accepted with {} resources: {}", type_url_,
resources.size(), RepeatedPtrUtil::debugString(typed_resources));
}
ENVOY_LOG(debug, "gRPC config for {} accepted with {} resources: {}", type_url_,
resources.size(), RepeatedPtrUtil::debugString(typed_resources));
}

void onConfigUpdateFailed(const EnvoyException* e) override {
Expand Down

0 comments on commit 4843fcc

Please sign in to comment.