David Knupp has posted comments on this change. Change subject: IMPALA-3764: fuzz test HDFS scanners ......................................................................
Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/3448/5/tests/common/impala_test_suite.py File tests/common/impala_test_suite.py: PS5, Line 514: self.execute_query_using_client I'm curious -- what's the difference between calling this method and asserting 0 at the end in case 'Location' is not found VS. (for example) using execute_query_expect_success? Is there a possibility that result.data could contain something, just not the location? -- To view, visit http://gerrit.cloudera.org:8080/3448 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I50cf43195a7c582caa02c85ae400ea2256fa3a3b Gerrit-PatchSet: 5 Gerrit-Project: Impala Gerrit-Branch: cdh5-trunk Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: David Knupp <[email protected]> Gerrit-Reviewer: Taras Bobrovytsky <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-HasComments: Yes
