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