Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8048
to look at the new patch set (#2).
Change subject: KUDU-2135 (part 1): add persistent disk states
......................................................................
KUDU-2135 (part 1): add persistent disk states
This patch introduces a new format for PathInstanceMetadataFiles that
includes a disk health and path for each data directory, and a timestamp
that the path instance file was last updated.
These additions will be necessary to ensure failed disks will not be
used and that data is where expect it to be (which will be increasingly
important as we begin to supporting removal/addition/replacement of
disks).
The expected usage of the new format would enforce startup restrictions
on the server (e.g. to not use a failed disk, to not use data if it is
on a different disk than expected, etc.) and tooling does not yet exist
to work around these restrictions. As such, this patch will upgrade and
maintain the new path sets as necessary, persisting disk health when
disks fail, but will not apply the mentioned restrictions at startup.
A test is added to data_dirs-test to ensure the path instance files get
updated/upgraded appropriately.
Change-Id: Ia7ea6418f6190d7d0c6d5a1aaca4ca592c49ce41
---
M src/kudu/fs/block_manager_util-test.cc
M src/kudu/fs/block_manager_util.cc
M src/kudu/fs/block_manager_util.h
M src/kudu/fs/data_dirs-test.cc
M src/kudu/fs/data_dirs.cc
M src/kudu/fs/data_dirs.h
M src/kudu/fs/fs.proto
7 files changed, 470 insertions(+), 76 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/8048/2
--
To view, visit http://gerrit.cloudera.org:8080/8048
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia7ea6418f6190d7d0c6d5a1aaca4ca592c49ce41
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
