Hello Alexey Serbin, Andrew Wong,

I'd like you to do a code review. Please visit

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

to review the following change.


Change subject: [tests] Add iteration to test dir path
......................................................................

[tests] Add iteration to test dir path

GTest supports running tests repeatedly by setting a "--gtest_repeat"
flag, which can be incredibly useful to debug flaky tests when using
together with "--gtest_break_on_failure" in gdb.

Unfortunately, most of our tests expect an empty data directory, but the
test directory was the same across repeated runs without being emptied.

Adding the iteration number to the end of the directory solves this
problem and --gtest_repeat can be used safely.

Change-Id: I175756e6f758246f2234eae3a0ad99a544f80026
---
M src/kudu/util/test_main.cc
M src/kudu/util/test_util.cc
M src/kudu/util/test_util.h
3 files changed, 23 insertions(+), 7 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I175756e6f758246f2234eae3a0ad99a544f80026
Gerrit-Change-Number: 18030
Gerrit-PatchSet: 1
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>

Reply via email to