Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20802 )
Change subject: IMPALA-10048: Go parallel for dump_breakpad_symbols.py ...................................................................... IMPALA-10048: Go parallel for dump_breakpad_symbols.py This modifies dump_breakpad_symbols.py to use a ThreadPool to go parallel when there are multiple binaries or libraries to process. This is common for Jenkins jobs that dump symbols for all backend tests. The different binaries write out to different directories, so the threads don't interfere with each other. Testing: - Ran locally dumping the symbols for all backend tests - Ran a Jenkins job that generates a minidump and triggers the minidump symbol processing. It went parallel and worked fine. Change-Id: I93427bb07f1d9718bd6df90acfd247210b54294d Reviewed-on: http://gerrit.cloudera.org:8080/20802 Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Reviewed-by: Michael Smith <michael.sm...@cloudera.com> --- M bin/dump_breakpad_symbols.py 1 file changed, 17 insertions(+), 3 deletions(-) Approvals: Impala Public Jenkins: Verified Michael Smith: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/20802 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I93427bb07f1d9718bd6df90acfd247210b54294d Gerrit-Change-Number: 20802 Gerrit-PatchSet: 5 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
