Adar Dembo has submitted this change and it was merged. Change subject: block_manager: disk space checking everywhere ......................................................................
block_manager: disk space checking everywhere This patch extends the disk space checking feature to the file block manager and to the log block manager without preallocation. Neither of these are hotly requested, but it's easy to do and it removes all conditions from the help text, so I think it's a net benefit. Change-Id: Ibbce30d7fa981255949ade23373c13939b1e3d43 Reviewed-on: http://gerrit.cloudera.org:8080/4832 Tested-by: Kudu Jenkins Reviewed-by: Mike Percy <[email protected]> --- M src/kudu/fs/block_manager-test.cc M src/kudu/fs/data_dirs.cc M src/kudu/fs/file_block_manager.cc M src/kudu/fs/log_block_manager.cc M src/kudu/integration-tests/disk_reservation-itest.cc 5 files changed, 52 insertions(+), 63 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4832 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibbce30d7fa981255949ade23373c13939b1e3d43 Gerrit-PatchSet: 6 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
