Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/18714
Change subject: KUDU-2671 fix double-free in new scenario of scan_token-test ...................................................................... KUDU-2671 fix double-free in new scenario of scan_token-test Changelist b746978c7 introduced a bug in the updated CountRowsSeq() function -- the mistake was calling delete twice on KuduScanToken pointers. That lead to reading garbage data, so the newly introduced test scenario ScanTokensWithCustomHashSchemasPerRange was failing if enabled. I didn't pay enough attention to the updated test code since the new test scenario was disabled in b746978c7. This patch fixes the mistake. Change-Id: I227f1dadf9d5b4d3b209570716cde7bda74c6b25 --- M src/kudu/client/scan_token-test.cc 1 file changed, 12 insertions(+), 11 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/14/18714/1 -- To view, visit http://gerrit.cloudera.org:8080/18714 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I227f1dadf9d5b4d3b209570716cde7bda74c6b25 Gerrit-Change-Number: 18714 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
