Tim Armstrong has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11879 )

Change subject: IMPALA-7477: Batch-oriented query set construction
......................................................................


Patch Set 1:

(1 comment)

This is a clean cherry-pick of https://gerrit.cloudera.org/#/c/11297/ with the 
required fix that I commented on in hs2-util.cc. IMPALA-7647 addressed the test 
gap by running more queries for a variety of data types through HS2.

http://gerrit.cloudera.org:8080/#/c/11879/1/be/src/service/hs2-util.cc
File be/src/service/hs2-util.cc:

http://gerrit.cloudera.org:8080/#/c/11879/1/be/src/service/hs2-util.cc@149
PS1, Line 149:     ++output_row_idx;
++output_row_idx here and below was the required fix.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I764fa302842438902cd5db2551ec6e3cb77b6874
Gerrit-Change-Number: 11879
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Mon, 05 Nov 2018 20:22:55 +0000
Gerrit-HasComments: Yes

Reply via email to