Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/9737
Change subject: tools: return error when updating with no block manager instances ...................................................................... tools: return error when updating with no block manager instances Previously when running `kudu fs update_dirs` with a set of directories that did not contain existing block manager instances, the tool would fail a CHECK. This CHECK has been around for a while for safety, but after a series of changes (refactors to support updating directories, configuration of the metadata directory, etc.), the premise that block manager instances are always present has been lost. Now, if Kudu can't find any block manager instances, an error is returned instead: Not found: cannot update data directories: could not open directory manager; none of the provided data directories could be found. Note that this only affects the `kudu fs update_dirs` tool; "normal" opening of the FS layout would return errors if any block manager instances are not found. Change-Id: I06d95f561301eebc21542e01782a8c219ca71c70 --- M src/kudu/fs/data_dirs.cc M src/kudu/tools/kudu-tool-test.cc 2 files changed, 38 insertions(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/37/9737/1 -- To view, visit http://gerrit.cloudera.org:8080/9737 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I06d95f561301eebc21542e01782a8c219ca71c70 Gerrit-Change-Number: 9737 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <[email protected]>
