Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/6758
Change subject: env: Always read fully when reading files ...................................................................... env: Always read fully when reading files In KUDU-9(see ) env_util::ReadFully was added to ensure short reads were retried until all data was read. (see a15c795360e32885c00442efacd2a345f993f425) Later RWFile was implimented with “read fully” behavior by default. (see a15c795360e32885c00442efacd2a345f993f425) Given we almost always use ReadFully or expect the data to be fully written regardles, this patch moves the “read fully” behavior into the Read function so it is always used. Change-Id: I735cd4cfca3c355226266ef4f0fdb57bf59dfe69 --- M src/kudu/consensus/log_util.cc M src/kudu/fs/file_block_manager.cc M src/kudu/fs/log_block_manager-test.cc M src/kudu/tserver/tablet_copy_client-test.cc M src/kudu/tserver/tablet_copy_service-test.cc M src/kudu/tserver/tablet_copy_source_session.cc M src/kudu/tserver/tablet_copy_source_session.h M src/kudu/util/env-test.cc M src/kudu/util/env_posix.cc M src/kudu/util/env_util.cc M src/kudu/util/env_util.h M src/kudu/util/pb_util-test.cc M src/kudu/util/pb_util.cc 13 files changed, 84 insertions(+), 126 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/58/6758/1 -- To view, visit http://gerrit.cloudera.org:8080/6758 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I735cd4cfca3c355226266ef4f0fdb57bf59dfe69 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <granthe...@gmail.com>
