Skip to content

Commit

Permalink
Make sure to combine hash values in the same way as the other row_has…
Browse files Browse the repository at this point in the history
…her does
  • Loading branch information
magnatelee committed Apr 1, 2021
1 parent 429658c commit 7113d6a
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions cpp/include/cudf/table/row_operators.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -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_value_type>{}.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<hash_function, has_nulls>{_seed},
_table.column(0),
row_index);
hash_combiner(hash_value_type{0},
type_dispatcher(_table.column(0).type(),
element_hasher_with_seed<hash_function, has_nulls>{_seed},
_table.column(0),
row_index));

// Hashes an element in a column
auto hasher = [=](size_type column_index) {
Expand Down

0 comments on commit 7113d6a

Please sign in to comment.