tree 1da62abb63934ec217edb4de880c44fe18029e23
parent 74dd9e9bfe32541e9ab6bddb8b839f3e491361f1
author mmokhtar <mmokhtar@cloudera.com> 1517879102 -0800
committer Impala Public Jenkins <impala-public-jenkins@gerrit.cloudera.org> 1519251082 +0000

IMPALA-6461 : Micro-optimizations to DataStreamSender::Send

While analyzing performance of partition exchange operator,
I noticed that there is dependency and a function call per row in the hot path.

Optimizations in this change are:
1) Remove the data dependency between computing the hash and the channel
2) Inline DataStreamSender::Channel::AddRow
3) Save partition_exprs_.size() to save a couple of instructions

This translates to improving CPI for DataStreamSender::Send by 10%

Change-Id: I642a9dad531a29d4838a3537ab0e04320a69960d
Reviewed-on: http://gerrit.cloudera.org:8080/9221
Reviewed-by: Mostafa Mokhtar <mmokhtar@cloudera.com>
Tested-by: Impala Public Jenkins