[ENHANCEMENT]: Consider removing the use of `argument_type` and `result_type` from `hasher` in cuco `default_filter_policy`
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Cuda
- Estrellas
- 667
- Forks
- 120
- Merge medio
- 7 d 5 h
- PR fusionados (30 d)
- 4
Descripción
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
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Busca en cuCollections hasher::argument_type y hasher::result_type, y después inspecciona su uso en las políticas de filtros Bloom. Identifica los hashers y las políticas afectados antes de cambiar nada. La tarea estará terminada cuando los alias obsoletos y sus usos se hayan eliminado en cuCollections, y las compilaciones y pruebas relevantes sigan siendo exitosas.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- backend
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100