[kudu-CR] dist test: add a cache for ldd results
Todd Lipcon has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9483 ) Change subject: dist_test: add a cache for ldd results .. dist_test: add a cache for ldd results With ctest-based sharding we ended up running 'ldd' many times on the same executable, which isn't necessary. This adds a simple cache for the results of ldd. Change-Id: I350ce78d61d9f4428424350ea6b41d4e61e3a4ae Reviewed-on: http://gerrit.cloudera.org:8080/9483 Reviewed-by: Adar DemboTested-by: Kudu Jenkins --- M build-support/dist_test.py 1 file changed, 7 insertions(+), 4 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9483 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I350ce78d61d9f4428424350ea6b41d4e61e3a4ae Gerrit-Change-Number: 9483 Gerrit-PatchSet: 2 Gerrit-Owner: Todd Lipcon Gerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon
[kudu-CR] dist test: add a cache for ldd results
Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/9483 ) Change subject: dist_test: add a cache for ldd results .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9483 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I350ce78d61d9f4428424350ea6b41d4e61e3a4ae Gerrit-Change-Number: 9483 Gerrit-PatchSet: 1 Gerrit-Owner: Todd LipconGerrit-Reviewer: Adar Dembo Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Sun, 04 Mar 2018 16:51:38 + Gerrit-HasComments: No
[kudu-CR] dist test: add a cache for ldd results
Hello Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9483 to review the following change. Change subject: dist_test: add a cache for ldd results .. dist_test: add a cache for ldd results With ctest-based sharding we ended up running 'ldd' many times on the same executable, which isn't necessary. This adds a simple cache for the results of ldd. Change-Id: I350ce78d61d9f4428424350ea6b41d4e61e3a4ae --- M build-support/dist_test.py 1 file changed, 7 insertions(+), 4 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/83/9483/1 -- To view, visit http://gerrit.cloudera.org:8080/9483 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I350ce78d61d9f4428424350ea6b41d4e61e3a4ae Gerrit-Change-Number: 9483 Gerrit-PatchSet: 1 Gerrit-Owner: Todd LipconGerrit-Reviewer: Adar Dembo