Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1904 Don't seek on empty RLE blocks ......................................................................
KUDU-1904 Don't seek on empty RLE blocks Seeking in RLE blocks enforces that the seek position is less than the number of elements in the block. If the number of elements is 0, as it is when the entire set of cells is NULL, this check will always result in a crash. This patch enforces that the block only seeks if there are elements in the block. Test coverage will be added in an upcoming commit. Change-Id: I6d6bd95099c9f30aa923a0da1e76f92ed90ebd99 Reviewed-on: http://gerrit.cloudera.org:8080/6184 Reviewed-by: David Ribeiro Alves <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/cfile/rle_block.h 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6184 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d6bd95099c9f30aa923a0da1e76f92ed90ebd99 Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
