Tim Armstrong 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 assert I actually just moved this method from a different file without changing it. I think the assert is just to catch the mistake if we somehow mess up the parsing. -- 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
