impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Ho (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3206: Enable codegen for AVRO DECIMAL
Date Fri, 24 Jun 2016 21:39:06 GMT
Michael Ho has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3489

Change subject: IMPALA-3206: Enable codegen for AVRO_DECIMAL
......................................................................

IMPALA-3206: Enable codegen for AVRO_DECIMAL

This change adds the missing switch statement in
CodegenReadScalar() for AVRO_DECIMAL so that we will
also codegen if an avro table contains AVRO_DECIMAL.
With this change, the following query improves by 37.5%,
going from 8s to 5s:

select count(distinct l_linenumber), avg(l_extendedprice), max(l_discount), min(l_tax) from
tpch15_avro.lineitem;

This change also un-inlines BitUtil::ByteSwap() as the
third argument 'len' is not compilation constant for
all call sites.

Change-Id: I51adf0c1ba76e055f31ccb0034a0d23ea2afb30e
---
M be/src/codegen/gen_ir_descriptions.py
M be/src/exec/hdfs-avro-scanner.cc
M be/src/util/CMakeLists.txt
R be/src/util/bit-util.cc
M be/src/util/bit-util.h
M be/src/util/decimal-util.h
6 files changed, 8 insertions(+), 10 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/89/3489/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3489
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I51adf0c1ba76e055f31ccb0034a0d23ea2afb30e
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <kwho@cloudera.com>

Mime
View raw message