Jean-Daniel Cryans 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/6201 Reviewed-by: Jean-Daniel Cryans <[email protected]> Tested-by: Jean-Daniel Cryans <[email protected]> --- M src/kudu/cfile/rle_block.h 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/6201 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d6bd95099c9f30aa923a0da1e76f92ed90ebd99 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
