Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14693 )
Change subject: [cfile] No longer need to `relocate()` in `ReadBlock()` ...................................................................... [cfile] No longer need to `relocate()` in `ReadBlock()` In the current implementation, when reading data from `ReadableBlock`, the `Slice` object will no longer be modified, but just fill it with data. That is, slice->data() will always match pre-allocated buffer. Change-Id: I1619d05d42c9ec20e029097aaaf37b3ac93b9a69 Reviewed-on: http://gerrit.cloudera.org:8080/14693 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/cfile/cfile_reader.cc 1 file changed, 0 insertions(+), 6 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/14693 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1619d05d42c9ec20e029097aaaf37b3ac93b9a69 Gerrit-Change-Number: 14693 Gerrit-PatchSet: 2 Gerrit-Owner: Anonymous Coward <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
