Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23593 )
Change subject: KUDU-1261 correct estimate for validity bitmap encoder buffer ...................................................................... KUDU-1261 correct estimate for validity bitmap encoder buffer Since the estimate should be for the maximum possible size, it should assume just a single element per array, not the maximum number of elements, of course. This changelist corrects the estimation. This is a follow-up to 18b1930e830609248ee2e26d6216d073ff40ac5d. Change-Id: Id7fc3c28043be160daa03372009117f5bfaf3d5b Reviewed-on: http://gerrit.cloudera.org:8080/23593 Reviewed-by: Abhishek Chennaka <[email protected]> Tested-by: Alexey Serbin <[email protected]> --- M src/kudu/cfile/cfile_writer.cc 1 file changed, 1 insertion(+), 5 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/23593 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id7fc3c28043be160daa03372009117f5bfaf3d5b Gerrit-Change-Number: 23593 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
