Hello Andrew Wong,

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

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

to review the following change.


Change subject: test_util: reset flags before cleaning test directory
......................................................................

test_util: reset flags before cleaning test directory

TabletServerDiskFailureTest.TestRandomOpSequence didn't reset
env_inject_eio to 0 before finishing. Rather than fixing that, let's change
KuduTest to reset its flags _before_ cleaning out the test directory. This
obviates the need for all such workarounds.

Change-Id: I4b4843cbc2c3bc2cbeb3a4d5959f80a35da7714a
---
M src/kudu/fs/block_manager-test.cc
M src/kudu/fs/data_dirs-test.cc
M src/kudu/fs/fs_manager-test.cc
M src/kudu/fs/log_block_manager-test.cc
M src/kudu/tablet/diskrowset-test.cc
M src/kudu/util/test_util.cc
M src/kudu/util/test_util.h
7 files changed, 22 insertions(+), 18 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4b4843cbc2c3bc2cbeb3a4d5959f80a35da7714a
Gerrit-Change-Number: 9944
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>

Reply via email to