Adar Dembo has submitted this change and it was merged. Change subject: env: add ReadV() API ......................................................................
env: add ReadV() API Adds ReadV() methods to RWFile and RandomAccessFile that allows reading data into multiple Slices in one call. The implementation leverages the preadv system call when possible and simulates it with pread calls when unavailable. Additionally adds ReadV() methods to the block manager abstraction. These methods will be used in KUDU-463 to support reading checksums and block data in a single call. Change-Id: Ib8f7a62c8363b40baa064d9e63be1ece506f1e48 Reviewed-on: http://gerrit.cloudera.org:8080/6779 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/fs/block_manager-test.cc M src/kudu/fs/block_manager.h M src/kudu/fs/file_block_manager.cc M src/kudu/fs/fs-test-util.h M src/kudu/fs/log_block_manager.cc M src/kudu/util/env-test.cc M src/kudu/util/env.h M src/kudu/util/env_posix.cc M src/kudu/util/file_cache.cc 9 files changed, 326 insertions(+), 15 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6779 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib8f7a62c8363b40baa064d9e63be1ece506f1e48 Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot
