Csaba Ringhofer created IMPALA-8381:
---------------------------------------

             Summary: Remove branch from ParquetPlainEncoder::Decode()
                 Key: IMPALA-8381
                 URL: https://issues.apache.org/jira/browse/IMPALA-8381
             Project: IMPALA
          Issue Type: Improvement
          Components: Backend
            Reporter: Csaba Ringhofer


Removing the "if" at
https://github.com/apache/impala/blob/5670f96b828d57f9e36510bb9af02bcc31de775c/be/src/exec/parquet/parquet-common.h#L203
can lead to 1.5x speed up in plain decoding (type=int32, stride=16). For 
primitive types, the same check can be done for a whole batch, so the speedup 
can be gained for large batches without loosing safety. The only Parquet type 
where this check is needed per element is BYTE_ARRAY (typically used for STRING 
columns), which already has a template specialization for  
ParquetPlainEncoder::Decode().




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to