Sahil Takiar has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16041 )

Change subject: IMPALA-9818: Add fetch size as option to impala shell
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16041/2/shell/option_parser.py
File shell/option_parser.py:

http://gerrit.cloudera.org:8080/#/c/16041/2/shell/option_parser.py@287
PS2, Line 287:   parser.add_option("--fetch_size", type="int", 
dest="fetch_size", default=1024,
> A higher number (10000?, similar to the JDBC driver) is probably a better d
Changed to 10240, which is 10x the batch size. This is actually the default 
fetch size if none is specified. Making it divisible by the default batch size 
has advantages of fetch performance in the result spooling implementation.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8dc7962aada6b38795241d067a99bd94fabca57b
Gerrit-Change-Number: 16041
Gerrit-PatchSet: 2
Gerrit-Owner: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Sahil Takiar <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Comment-Date: Tue, 09 Jun 2020 20:12:37 +0000
Gerrit-HasComments: Yes

Reply via email to