Taras Bobrovytsky has posted comments on this change.

Change subject: Reuse session for executing queries (Hive on Spark)
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2374/2/tests/performance/query_exec_functions.py
File tests/performance/query_exec_functions.py:

Line 70:     if cursor is not None:
I don't think it's necessary to check if it's None. Just go ahead and attach it 
to the current thread.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I06c798dc311d63eb0a875450fd26d06db4e84a03
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Kapil Rastogi <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Ishaan Joshi <[email protected]>
Gerrit-Reviewer: Kapil Rastogi <[email protected]>
Gerrit-Reviewer: Taras Bobrovytsky <[email protected]>
Gerrit-HasComments: Yes

Reply via email to