[kudu-CR](branch-1.2.x) KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks
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 AlvesReviewed-by: Andrew Wong (cherry picked from commit d1f53cc323d5d07e79304765fe171f535c1d548a) Reviewed-on: http://gerrit.cloudera.org:8080/7613 --- M src/kudu/cfile/binary_prefix_block.cc M src/kudu/cfile/encoding-test.cc 2 files changed, 51 insertions(+), 10 deletions(-) Approvals: David Ribeiro Alves: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/7613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1670b2244d586e4920f0603c48f65ed993a3b12b Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Todd Lipcon Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR](branch-1.2.x) KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/7613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1670b2244d586e4920f0603c48f65ed993a3b12b Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Todd LipconGerrit-Reviewer: Andrew Wong Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: No
[kudu-CR](branch-1.2.x) KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks
Todd Lipcon has posted comments on this change. Change subject: KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/7613/2/src/kudu/cfile/encoding-test.cc File src/kudu/cfile/encoding-test.cc: Line 260: const auto& GenTestString = [&](int x) -> string { in branch-1.2.x I had to add this explicit return type, because the older version of gtest wasn't happy doing all the type deduction below in the asserts. -- To view, visit http://gerrit.cloudera.org:8080/7613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I1670b2244d586e4920f0603c48f65ed993a3b12b Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Todd LipconGerrit-Reviewer: Andrew Wong Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon Gerrit-HasComments: Yes
[kudu-CR](branch-1.2.x) KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks
Hello David Ribeiro Alves, Andrew Wong, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/7613 to look at the new patch set (#2). 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 AlvesReviewed-by: Andrew Wong (cherry picked from commit d1f53cc323d5d07e79304765fe171f535c1d548a) --- M src/kudu/cfile/binary_prefix_block.cc M src/kudu/cfile/encoding-test.cc 2 files changed, 51 insertions(+), 10 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/7613/2 -- To view, visit http://gerrit.cloudera.org:8080/7613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I1670b2244d586e4920f0603c48f65ed993a3b12b Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Todd Lipcon Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins
[kudu-CR](branch-1.2.x) KUDU-2085. Fix crash when seeking past end of prefix-encoded blocks
Hello David Ribeiro Alves, Andrew Wong, Kudu Jenkins, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/7613 to review the following change. 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 AlvesReviewed-by: Andrew Wong (cherry picked from commit d1f53cc323d5d07e79304765fe171f535c1d548a) --- M src/kudu/cfile/binary_prefix_block.cc M src/kudu/cfile/encoding-test.cc 2 files changed, 50 insertions(+), 10 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/7613/1 -- To view, visit http://gerrit.cloudera.org:8080/7613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I1670b2244d586e4920f0603c48f65ed993a3b12b Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: branch-1.2.x Gerrit-Owner: Todd Lipcon Gerrit-Reviewer: Andrew Wong Gerrit-Reviewer: David Ribeiro Alves Gerrit-Reviewer: Kudu Jenkins