[ENHANCEMENT]: Consider removing the use of `argument_type` and `result_type` from `hasher` in cuco `default_filter_policy`
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Cuda
- Sterne
- 667
- Forks
- 120
- Ø Merge
- 7 T. 5 Std.
- Gemergte PRs (30 T.)
- 4
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Durchsuche cuCollections nach hasher::argument_type und hasher::result_type und untersuche anschließend deren Verwendung in Bloom-Filter-Richtlinien. Erfasse die betroffenen Hasher und Richtlinien, bevor du irgendetwas änderst. Als erledigt gilt die Aufgabe, wenn die veralteten Aliase und ihre Verwendungen in cuCollections entfernt sind und die relevanten Builds und Tests weiterhin erfolgreich sind.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- cpp
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100