Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11863 )
Change subject: python: make test_scantoken more robust to exceptions ...................................................................... python: make test_scantoken more robust to exceptions One of my precommit tests failed with: 22:49:05 kudu/tests/test_client.py ............................. 22:49:06 kudu/tests/test_scanner.py ........................ 22:49:08 kudu/tests/test_scantoken.py ..Build timed out (after 50 minutes). Marking the build as failed. 23:39:08 Build was aborted I don't know why this happened, but looking at commit 1d928951e, it seems like if pool.map() raised an exception, we wouldn't close or join the pool, which would lead to the same test hanging behavior as before. I don't know how or why pool.map() raised an exception, but this will hopefully surface the actual root cause. Change-Id: I9b5a70d6b5f115cc4de740ded6ecc48574b9c707 Reviewed-on: http://gerrit.cloudera.org:8080/11863 Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M python/kudu/tests/test_scantoken.py 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11863 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9b5a70d6b5f115cc4de740ded6ecc48574b9c707 Gerrit-Change-Number: 11863 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Jordan Birdsell <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
