From 7113d6a2697a5626f992a4d84c142022cd85d530 Mon Sep 17 00:00:00 2001 From: Wonchan Lee Date: Wed, 31 Mar 2021 23:37:38 -0700 Subject: [PATCH] Make sure to combine hash values in the same way as the other row_hasher does --- cpp/include/cudf/table/row_operators.cuh | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/cpp/include/cudf/table/row_operators.cuh b/cpp/include/cudf/table/row_operators.cuh index b78de25afb1..61d714c5538 100644 --- a/cpp/include/cudf/table/row_operators.cuh +++ b/cpp/include/cudf/table/row_operators.cuh @@ -468,16 +468,17 @@ class row_hasher { __device__ auto operator()(size_type row_index) const { - auto hash_combiner = [=](hash_value_type lhs, hash_value_type rhs) { + auto hash_combiner = [](hash_value_type lhs, hash_value_type rhs) { return hash_function{}.hash_combine(lhs, rhs); }; // Hash the first column w/ the seed auto const initial_hash = - type_dispatcher(_table.column(0).type(), - element_hasher_with_seed{_seed}, - _table.column(0), - row_index); + hash_combiner(hash_value_type{0}, + type_dispatcher(_table.column(0).type(), + element_hasher_with_seed{_seed}, + _table.column(0), + row_index)); // Hashes an element in a column auto hasher = [=](size_type column_index) {