Mike Percy has submitted this change and it was merged. Change subject: env: Add support for getting FS capacity ......................................................................
env: Add support for getting FS capacity We need a way to get FS capacity in a follow-up patch. This change adds that capability and changes the Env API to allow for fetching both capacity and free bytes with a single call. This also allows us to fetch both values with a single syscall. This API is inspired by boost::filesystem::space(). This patch also removes the difference in the "free" space returned when this API is invoked as a superuser vs. a non-privileged user. Now, only the space available to non-privileged processes is returned. This is also consistent with the boost API and makes the API more predictable. Change-Id: Id43275876d3352f5cf943e24ed4513b9f2c131aa Reviewed-on: http://gerrit.cloudera.org:8080/6255 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <[email protected]> Reviewed-by: David Ribeiro Alves <[email protected]> --- M src/kudu/util/env-test.cc M src/kudu/util/env.h M src/kudu/util/env_posix.cc M src/kudu/util/env_util.cc 4 files changed, 49 insertions(+), 32 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Todd Lipcon: Looks good to me, but someone else must approve Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6255 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id43275876d3352f5cf943e24ed4513b9f2c131aa Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
