Matthew Mulder has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8102 )

Change subject: IMPALA-5376: Implement all TPCDS test cases or alternates for 
Impala.
......................................................................


Patch Set 10:

I found a bug in the stress test. There are already queries named like 
tpcds-q23-1.test or tpcds-q27a.test that won't be picked up by the stress test 
because of this regex:
  file_name_pattern = re.compile(r"%s-%s(q\d+).test$" % (workload, engine))
I'll file a bug (if there isn't one already), but thought I'd share this so you 
don't wonder why some of your new queries are skipped by the stress test.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6e284888600a7a69d1f23fcb7dac21cbb13b7d66
Gerrit-Change-Number: 8102
Gerrit-PatchSet: 10
Gerrit-Owner: Tim Wood <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: David Knupp <[email protected]>
Gerrit-Reviewer: Matthew Mulder <[email protected]>
Gerrit-Reviewer: Michael Brown <[email protected]>
Gerrit-Reviewer: Michael Ho
Gerrit-Reviewer: Mostafa Mokhtar <[email protected]>
Gerrit-Reviewer: Tim Wood <[email protected]>
Gerrit-Comment-Date: Fri, 29 Sep 2017 00:19:46 +0000
Gerrit-HasComments: No

Reply via email to