Hello Jordan Birdsell, Andrew Wong,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/11863
to review the following change.
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
---
M python/kudu/tests/test_scantoken.py
1 file changed, 5 insertions(+), 3 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/63/11863/1
--
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: newchange
Gerrit-Change-Id: I9b5a70d6b5f115cc4de740ded6ecc48574b9c707
Gerrit-Change-Number: 11863
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Jordan Birdsell <[email protected]>
