[ENHANCEMENT]: Consider removing the use of `argument_type` and `result_type` from `hasher` in cuco `default_filter_policy`
Personne n'a encore pris cette issue.
- Langage dominant
- Cuda
- Étoiles
- 667
- Forks
- 120
- Merge moyen
- 7 j 5 h
- PR mergées (30 j)
- 4
Description
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
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Recherchez hasher::argument_type et hasher::result_type dans cuCollections, puis examinez leur utilisation dans les politiques de filtres de Bloom. Identifiez les hashers et les politiques concernés avant de modifier quoi que ce soit. La tâche est terminée lorsque les alias obsolètes et leurs utilisations ont été supprimés dans cuCollections et que les builds et les tests concernés restent concluants.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cpp
- Domaine
- backend
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100