You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Todd Lipcon (Code Review)" <ge...@ip-10-146-233-104.ec2.internal> on 2016/01/08 04:12:33 UTC

[kudu-CR] Fix mem_tracker-test failures on clang 3.7 -O3

Todd Lipcon has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/1750

Change subject: Fix mem_tracker-test failures on clang 3.7 -O3
......................................................................

Fix mem_tracker-test failures on clang 3.7 -O3

clang 3.7 running with -O3 has an optimization that removes unused
memory allocations. One of the mem_tracker-tests relies on making such
an allocation to artificially increase the heap usage, so the test would
fail.

This adds a useless log message to log the allocated pointer, defeating
the optimization.

Change-Id: Ic1015f93e65962ffa78062a83c20a643373810bc
---
M src/kudu/util/mem_tracker-test.cc
1 file changed, 3 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/1750/1
-- 
To view, visit http://gerrit.cloudera.org:8080/1750
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic1015f93e65962ffa78062a83c20a643373810bc
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@cloudera.com>