Michael Brown has posted comments on this change.

Change subject: IMPALA-3957: Test failure in S3 build: TestLoadData.test_load
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3881/1/tests/util/s3_util.py
File tests/util/s3_util.py:

PS1, Line 48:     assert self.exists(dst)
The failure will be a little more obvious easier to spot if you supply a second 
argument to "assert" explaining the failure, like:

  assert self.exists(dst), 's3 copy failed: destination file {dst} does not 
exist'.format(dst=dst)


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I966a469e94099d3d971e470ae6e992386070c5e9
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Michael Brown <[email protected]>
Gerrit-HasComments: Yes

Reply via email to