David Knupp has posted comments on this change. ( http://gerrit.cloudera.org:8080/11447 )
Change subject: IMPALA-7576: Add a timeout for all E2E tests ...................................................................... Patch Set 4: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/11447/4/tests/conftest.py File tests/conftest.py: http://gerrit.cloudera.org:8080/#/c/11447/4/tests/conftest.py@58 PS4, Line 58: """ Hook startup of pytest to set up log format. """ Moving to conftest.py was a great suggestion. Nit: this is docstring should be updated now. -- To view, visit http://gerrit.cloudera.org:8080/11447 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I301dd27a9767bfaef2756282014ef457a31956bd Gerrit-Change-Number: 11447 Gerrit-PatchSet: 4 Gerrit-Owner: Thomas Marshall <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Thomas Marshall <[email protected]> Gerrit-Comment-Date: Mon, 17 Sep 2018 17:45:58 +0000 Gerrit-HasComments: Yes
