NVIDIA / NVIDIA/cuCollections

[ENHANCEMENT]: Consider removing the use of `argument_type` and `result_type` from `hasher` in cuco `default_filter_policy`

Đang mở
#653 5 bình luận 0 reaction 0 người được giao Xem trên GitHub

Chưa có ai nhận issue này.

P1: Should have topic: bloom_filter type: improvement
Ngôn ngữ chính
Cuda
Star
667
Fork
120
Merge trung bình
7 ngày 5 giờ
Pull request đã merge (30 ngày)
4

Mô tả

Is your feature request related to a problem? Please describe.

The member type std::hash::argument_type is deprecated in C++17 and removed in C++20, so we should consider syncing with STL and remove them from our hashers as well as their use in bloom filter policies.

Ref: https://en.cppreference.com/w/cpp/utility/hash

Originally posted by @bdice in https://github.com/rapidsai/cudf/pull/17289#discussion_r1876965059

Describe the solution you'd like

Consider refactoring to not use hasher::argument_type and hasher::result_type across cuCollections.

Describe alternatives you've considered

Keep using the deprecated hasher type aliases until C++20

Additional context

No response

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Hướng nghiên cứu

Tìm kiếm hasher::argument_type và hasher::result_type trong cuCollections, sau đó kiểm tra việc sử dụng chúng trong các policy của bloom filter. Xác định các hasher và policy bị ảnh hưởng trước khi thay đổi bất cứ điều gì. Hoàn thành khi các alias đã lỗi thời và các chỗ sử dụng chúng được loại bỏ trong cuCollections, đồng thời các build và test liên quan vẫn thành công.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
cpp
Lĩnh vực
backend
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.