Marton Greber has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21670 )
Change subject: [cfile] modernize CFileWriter ...................................................................... [cfile] modernize CFileWriter * use PREDICT_FALSE for data corruption conditions * fix const-correctness of the CFileWriter::block() method * other minor updates This patch doesn't contain any functional modifications. Change-Id: I6038de39f912b054320b97af11145627c68ba00a Reviewed-on: http://gerrit.cloudera.org:8080/21670 Reviewed-by: Zoltan Martonka <[email protected]> Tested-by: Marton Greber <[email protected]> Reviewed-by: Marton Greber <[email protected]> --- M src/kudu/cfile/cfile_writer.cc M src/kudu/cfile/cfile_writer.h 2 files changed, 77 insertions(+), 82 deletions(-) Approvals: Zoltan Martonka: Looks good to me, but someone else must approve Marton Greber: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/21670 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6038de39f912b054320b97af11145627c68ba00a Gerrit-Change-Number: 21670 Gerrit-PatchSet: 4 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]> Gerrit-Reviewer: Zoltan Chovan <[email protected]> Gerrit-Reviewer: Zoltan Martonka <[email protected]>
