Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks ......................................................................
KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks This fixes a bug when seeking past the last element of a prefix-encoded binary block. In very specific circumstances, described in the JIRA, this would cause a Status::Corruption() to be returned. Prior to the fix, the updated test would fail pretty reliably when looped 50-100 times. With the patch, I looped it 1000 times with no failure. Change-Id: I1670b2244d586e4920f0603c48f65ed993a3b12b Reviewed-on: http://gerrit.cloudera.org:8080/7545 Tested-by: Kudu Jenkins Reviewed-by: David Ribeiro Alves <davidral...@gmail.com> Reviewed-by: Andrew Wong <aw...@cloudera.com> (cherry picked from commit d1f53cc323d5d07e79304765fe171f535c1d548a) Reviewed-on: http://gerrit.cloudera.org:8080/7611 Reviewed-by: Jean-Daniel Cryans <jdcry...@apache.org> Tested-by: Todd Lipcon <t...@apache.org> --- M src/kudu/cfile/binary_prefix_block.cc M src/kudu/cfile/encoding-test.cc 2 files changed, 48 insertions(+), 9 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Todd Lipcon: Verified -- To view, visit http://gerrit.cloudera.org:8080/7611 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1670b2244d586e4920f0603c48f65ed993a3b12b Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: branch-1.4.x Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>