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

Change subject: IMPALA-8381: WIP - Optimize ParquetPlainEncoder::DecodeBatch() 
for simple types
......................................................................


Patch Set 2:

Nice results!

I think at some point we might want to switch to materialising dense arrays and 
transposing to the row format in TransferScratchTuples() as part of the same 
pass as predicate evaluation. In that case this would be very beneficial.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I57b7d2573bb6dfd038e581acb3bd8ea1565aa20d
Gerrit-Change-Number: 12985
Gerrit-PatchSet: 2
Gerrit-Owner: Daniel Becker <daniel.bec...@cloudera.com>
Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com>
Gerrit-Reviewer: Daniel Becker <daniel.bec...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Comment-Date: Thu, 11 Apr 2019 18:40:58 +0000
Gerrit-HasComments: No

Reply via email to