pgsql: Speed up Hash Join by making ExprStates support hashing - Mailing list pgsql-committers

From David Rowley
Subject pgsql: Speed up Hash Join by making ExprStates support hashing
Date
Msg-id E1sgDqB-000W0s-Us@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Speed up Hash Join by making ExprStates support hashing

Here we add ExprState support for obtaining a 32-bit hash value from a
list of expressions.  This allows both faster hashing and also JIT
compilation of these expressions.  This is especially useful when hash
joins have multiple join keys as the previous code called ExecEvalExpr on
each hash join key individually and that was inefficient as tuple
deformation would have only taken into account one key at a time, which
could lead to walking the tuple once for each join key.  With the new
code, we'll determine the maximum attribute required and deform the tuple
to that point only once.

Some performance tests done with this change have shown up to a 20%
performance increase of a query containing a Hash Join without JIT
compilation and up to a 26% performance increase when JIT is enabled and
optimization and inlining were performed by the JIT compiler.  The
performance increase with 1 join column was less with a 14% increase
with and without JIT.  This test was done using a fairly small hash
table and a large number of hash probes.  The increase will likely be
less with large tables, especially ones larger than L3 cache as memory
pressure is more likely to be the limiting factor there.

This commit only addresses Hash Joins, but lays expression evaluation
and JIT compilation infrastructure for other hashing needs such as Hash
Aggregate.

Author: David Rowley
Reviewed-by: Alexey Dvoichenkov <alexey@hyperplane.net>
Reviewed-by: Tels <nospam-pg-abuse@bloodgate.com>
Discussion: https://postgr.es/m/CAApHDvoexAxgQFNQD_GRkr2O_eJUD1-wUGm%3Dm0L%2BGc%3DT%3DkEa4g%40mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/adf97c1562380e02acd60dc859c289ed3a8352ee

Modified Files
--------------
src/backend/executor/execExpr.c       | 141 +++++++++++++++++++++++
src/backend/executor/execExprInterp.c | 110 ++++++++++++++++++
src/backend/executor/nodeHash.c       | 190 +++++++------------------------
src/backend/executor/nodeHashjoin.c   | 143 ++++++++++++++++++++----
src/backend/jit/llvm/llvmjit_expr.c   | 204 ++++++++++++++++++++++++++++++++++
src/include/executor/execExpr.h       |  24 ++++
src/include/executor/executor.h       |   7 ++
src/include/executor/hashjoin.h       |  12 --
src/include/executor/nodeHash.h       |   9 +-
src/include/nodes/execnodes.h         |  12 +-
10 files changed, 650 insertions(+), 202 deletions(-)


pgsql-committers by date:

Previous
From: Bruce Momjian
Date:
Subject: pgsql: doc: improve create/alter sequence CYCLE syntax
Next
From: Amit Kapila
Date:
Subject: pgsql: Log the conflicts while applying changes in logical replication.