Skip to content

Commit

Permalink
Get rid of sentinel namespace (rapidsai#4274)
Browse files Browse the repository at this point in the history
This PR gets rid of the use of the deprecated `cuco::sentinel` inline namespace.

It's required to bump the cuco version in rapidsai/rapids-cmake#569

Authors:
  - Yunsong Wang (https://github.com/PointKernel)

Approvers:
  - Seunghwa Kang (https://github.com/seunghwak)

URL: rapidsai#4274
  • Loading branch information
PointKernel authored Apr 17, 2024
1 parent d7d479b commit 45d0126
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion cpp/src/prims/key_store.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ class key_cuco_store_t {
rmm::mr::polymorphic_allocator<std::byte>(rmm::mr::get_current_device_resource()), stream);
cuco_store_ =
std::make_unique<cuco_set_type>(cuco_size,
cuco::sentinel::empty_key<key_t>{invalid_key},
cuco::empty_key<key_t>{invalid_key},
thrust::equal_to<key_t>{},
cuco::linear_probing<1, // CG size
cuco::murmurhash3_32<key_t>>{},
Expand Down
8 changes: 4 additions & 4 deletions cpp/src/prims/kv_store.cuh
Original file line number Diff line number Diff line change
Expand Up @@ -823,8 +823,8 @@ class kv_cuco_store_t {
if constexpr (std::is_arithmetic_v<value_t>) {
cuco_store_ =
std::make_unique<cuco_map_type>(cuco_size,
cuco::sentinel::empty_key<key_t>{invalid_key},
cuco::sentinel::empty_value<value_t>{invalid_value},
cuco::empty_key<key_t>{invalid_key},
cuco::empty_value<value_t>{invalid_value},
thrust::equal_to<key_t>{},
cuco::linear_probing<1, // CG size
cuco::murmurhash3_32<key_t>>{},
Expand All @@ -835,8 +835,8 @@ class kv_cuco_store_t {
} else {
cuco_store_ = std::make_unique<cuco_map_type>(
cuco_size,
cuco::sentinel::empty_key<key_t>{invalid_key},
cuco::sentinel::empty_value<size_t>{std::numeric_limits<size_t>::max()},
cuco::empty_key<key_t>{invalid_key},
cuco::empty_value<size_t>{std::numeric_limits<size_t>::max()},
thrust::equal_to<key_t>{},
cuco::linear_probing<1, // CG size
cuco::murmurhash3_32<key_t>>{},
Expand Down

0 comments on commit 45d0126

Please sign in to comment.