apache / apache/datasketches-cpp
Reorganization proposal
- Lenguaje dominante
- C++
- Estrellas
- 273
- Forks
- 88
- Merge medio
- 2 d 2 h
- PR fusionados (30 d)
- 8
Descripción
Hi there,
We are consider using datasketches in our C++ environment. The current organization, where there a bunch of folders, each with an `include/` directory, is a bit awkward for integrating into a foreign build system. Projects like [pybind11](https://github.com/pybind/pybind11/) put all of the headers in a single `include/` directory (perhaps with subdirectories underneath). Then there is just a single path to add as an include path.
Would you be open to a reorganization that moved around files to look a bit like this:
```
include/datasketches
count/
count_min.hpp
count_min_impl.hpp
common/
MurmurHash3.h
...
```
Thanks for your consideration.
Guía de contribución
Línea de trabajo
Revisa las carpetas de componentes existentes del repositorio y sus directorios include/ y compáralos después con la disposición propuesta de include/datasketches y el precedente de pybind11 enlazado en la issue. Antes de cambiar archivos, confirma que los maintainers hayan acordado una organización objetivo; la tarea estaría terminada cuando exista una disposición aceptada que proporcione una única ruta include para sistemas de compilación externos.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- cpp
- Área
- build-system
- Tipo de issue
- Refactorización
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 20/100