Tim Armstrong has posted comments on this change.

Change subject: IMPALA-3969: stress test: add option to (en|dis)able codegen
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/3887/1/tests/stress/concurrent_select.py
File tests/stress/concurrent_select.py:

Line 724:         cursor.execute(
> If we want additional query options, instead of a separate query option add
I feel like there will probably be more query options. E.g. multithreading. So 
this seems like a good approach to me.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iada041aace60c218a12178d8f1b9a68ff29de72e
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Michael Brown <[email protected]>
Gerrit-Reviewer: David Knupp <[email protected]>
Gerrit-Reviewer: Michael Brown <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: Yes

Reply via email to