Hello Dinesh Bhat, Lars Volker, Adar Dembo,

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

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

to review the following change.

Change subject: EMC: Don't reuse data dir for log dir
......................................................................

EMC: Don't reuse data dir for log dir

If we put any files into the log dir before initializing the FsManager,
the FsManager will refuse to start up. Let's just always devote the data
directory to the FsManager.

This change is needed in order to integrate Breakpad in a later change,
since creating the necessary directory structure for Breakpad minidump
files in the shared log/data directory at startup time confuses
FsManager when we ask it to then initialize the data directory.

Change-Id: Iba23c429989df524da51eb012a491766df06e955
---
M src/kudu/integration-tests/external_mini_cluster.cc
M src/kudu/integration-tests/external_mini_cluster.h
2 files changed, 91 insertions(+), 112 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Iba23c429989df524da51eb012a491766df06e955
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com>
Gerrit-Reviewer: Lars Volker <l...@cloudera.com>

Reply via email to