Michael Brown has posted comments on this change.

Change subject: IMPALA-3608: Updates pytest to allow multiple exception messages
......................................................................


Patch Set 2:

(1 comment)

> (6 comments)
 > The following sand box build is in progress but I will start a new
 > one:
 > http://sandbox.jenkins.cloudera.com/job/impala-private-build-and-test/3206/

I'm sorry; of course GVM would have caught parsing problems. I'm not sure why I 
made this point.

http://gerrit.cloudera.org:8080/#/c/3205/2//COMMIT_MSG
Commit Message:

Line 7: IMPALA-3608: Updates pytest to allow multiple exception messages
Sorry for this nit, but pytest is an open source testing framework. You're not 
updating pytest, but instead *Impala's* end-to-end-test framework, which 
happens to use pytest.


-- 
To view, visit http://gerrit.cloudera.org:8080/3205
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie6d81fd3ae601f565b575edfeefff7c5a6c07974
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Michael Ho <[email protected]>
Gerrit-Reviewer: Michael Brown <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-HasComments: Yes

Reply via email to