Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8010 )
Change subject: pb_util: avoid repeated stat() calls reading files ...................................................................... pb_util: avoid repeated stat() calls reading files This reduces the number of fstat syscalls while loading a host with 11M blocks from 29.3M to 147K. Note that this also changes Env to return EndOfFile when reading from disk rather than IOError as it used to. Change-Id: I27371800604bcb20bafae7946d3b3e84af094598 Reviewed-on: http://gerrit.cloudera.org:8080/8010 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/util/env-test.cc M src/kudu/util/env_posix.cc M src/kudu/util/pb_util-test.cc M src/kudu/util/pb_util.cc M src/kudu/util/pb_util.h 5 files changed, 62 insertions(+), 22 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8010 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I27371800604bcb20bafae7946d3b3e84af094598 Gerrit-Change-Number: 8010 Gerrit-PatchSet: 5 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
