Use actual bytesarray/memoryview instead of bytes concatenation
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 38/100
Línea de trabajo
Comienza en fluent/sender.py, en la línea 137, y sigue la ruta de buffering alrededor de self.pendings. Lee la implementación circundante de sender para comparar el enfoque propuesto de bytearray o memoryview con la concatenación actual. La tarea estará completada cuando los envíos almacenados en búfer ya no concatenen bytes repetidamente y se conserve el comportamiento existente del sender.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Bytes concatenation such as this is extremely wasteful and is never a good idea.
https://github.com/fluent/fluent-logger-python/blob/master/fluent/sender.py#L137
# buffering
if self.pendings:
self.pendings += bytes_
bytes_ = self.pendings
Python has mutable buffers that should be used to ensure that the objects aren't created needlessly putting pressure on GC and wasting CPU with double-copy concatenation.
- Lenguaje dominante
- Python
- Estrellas
- 457
- Forks
- 138
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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.
Más de fluent/fluent-logger-python
-
support of `asctime`Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
fluent/fluent-logger-python#210 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 25/100
fluent/fluent-logger-python#196 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 42/100
fluent/fluent-logger-python#195 · 1 comentario ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 45/100
fluent/fluent-logger-python#194 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
fluent/fluent-logger-python#193 · 3 comentarios ·
Todos los issues de fluent/fluent-logger-python
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
rpm-software-management/mock#1824 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
jpata/particleflow#520 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue hacktoberfest
Dificultad 1/5 Menos de una hora Aptitud para principiantes 78/100
gridhead/gi-loadouts#699 ·
Los mantenedores suelen responder en 13 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 86/100
FinanceFlash/unvibecode#206 ·
Los mantenedores suelen responder en 1 día