Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14243 )
Change subject: tools: don't open block manager when dumping UUID ...................................................................... tools: don't open block manager when dumping UUID The block manager can be expensive to open, even when opening it in read-only mode. If all a user wants is the server's UUID, all we really need is to read the instance metadata files. This patch updates the `kudu fs dump uuid` tool to do just that. This also moves some words around in the header to try clarifying what functions are important for startup. Change-Id: I0acf3b08fc13c169a52d4cfe847efd62bd30cc0a Reviewed-on: http://gerrit.cloudera.org:8080/14243 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/fs/fs_manager.cc M src/kudu/fs/fs_manager.h M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/tool_action_fs.cc 4 files changed, 55 insertions(+), 28 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14243 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I0acf3b08fc13c169a52d4cfe847efd62bd30cc0a Gerrit-Change-Number: 14243 Gerrit-PatchSet: 6 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241)
