Yifan Zhang has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/19526 )
Change subject: KUDU-3451 fix memory leak in scan_token-test
......................................................................
KUDU-3451 fix memory leak in scan_token-test
This patch adds ElementDeleters to the test "TestScanTokensWithQueryId"
to fix the scan token leaks.
Running the test “TestScanTokensWithQueryId” 1024 times:
- without this patch all of them failed,
- with this patch, none of them failed, all were succesful.
Change-Id: Ic9f47dc0842f415ab1725a8080d59595896dc57e
Reviewed-on: http://gerrit.cloudera.org:8080/19526
Reviewed-by: Attila Bukor <[email protected]>
Tested-by: Kudu Jenkins
Reviewed-by: Mahesh Reddy <[email protected]>
Reviewed-by: Yifan Zhang <[email protected]>
---
M src/kudu/client/scan_token-test.cc
1 file changed, 2 insertions(+), 0 deletions(-)
Approvals:
Attila Bukor: Looks good to me, approved
Kudu Jenkins: Verified
Mahesh Reddy: Looks good to me, but someone else must approve
Yifan Zhang: Looks good to me, approved
--
To view, visit http://gerrit.cloudera.org:8080/19526
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ic9f47dc0842f415ab1725a8080d59595896dc57e
Gerrit-Change-Number: 19526
Gerrit-PatchSet: 3
Gerrit-Owner: Marton Greber <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Wang Xixu <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
