Tim Armstrong has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/8172 )
Change subject: PREVIEW: IMPALA-5307: Part 4: copy out uncompressed text and seq ...................................................................... PREVIEW: IMPALA-5307: Part 4: copy out uncompressed text and seq This is the final patch for IMPALA-5307. Text and Seq scanners are converted to use the same approach as Avro. contains_tuple_data is now false so a bunch of dead code in ScannerContext can be removed. We also no longer attach I/O buffers to row batches so that logic can be removed. Testing: Ran core ASAN tests. Perf: TODO - need to run numbers Change-Id: I304fd002b61bfedf41c8b1405cd7eb7b492bb941 --- M be/src/codegen/gen_ir_descriptions.py M be/src/exec/base-sequence-scanner.cc M be/src/exec/exec-node.cc M be/src/exec/exec-node.h M be/src/exec/hdfs-avro-scanner.cc M be/src/exec/hdfs-avro-scanner.h M be/src/exec/hdfs-parquet-scanner.cc M be/src/exec/hdfs-rcfile-scanner.cc M be/src/exec/hdfs-scan-node-mt.cc M be/src/exec/hdfs-scan-node.cc M be/src/exec/hdfs-scanner-ir.cc M be/src/exec/hdfs-scanner.cc M be/src/exec/hdfs-scanner.h M be/src/exec/hdfs-sequence-scanner.cc M be/src/exec/hdfs-text-scanner.cc M be/src/exec/parquet-column-readers.cc M be/src/exec/scanner-context.cc M be/src/exec/scanner-context.h M be/src/runtime/row-batch.cc M be/src/runtime/row-batch.h 20 files changed, 118 insertions(+), 192 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/72/8172/2 -- To view, visit http://gerrit.cloudera.org:8080/8172 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I304fd002b61bfedf41c8b1405cd7eb7b492bb941 Gerrit-Change-Number: 8172 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>