AzureDocumentDB: ConventionRegistry gets a new entry for every collection
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 76/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- csharp, mongodb
- Área
- database, performance
Línea de trabajo
Lee el constructor que llama a ConventionRegistry.Register y la ruta de GetCollection que crea instancias de colecciones. El issue señala mongodb/mongo-mevd-provider#9 como ejemplo de cómo trasladar el registro a un constructor estático; añade una prueba para la creación repetida de colecciones y verifica que el registro ya no añade una entrada cada vez.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I noticed MongoMapper calls ConventionRegistry.Register in the constructor. So every new collection (GetCollection creates a new one every time) adds one more entry to ConventionRegistry, and they are never removed.
I tried to create collection 100k times in a loop and after it ConventionRegistry had ~100k entries, around 35 MB of memory.
Looks like MongoDB already fixed this in their copy of this code (mongodb/mongo-mevd-provider#9), they moved it to a static constructor. I can do the same change here with a test, if its ok.
- Lenguaje dominante
- C#
- Estrellas
- 8
- Forks
- 8
- Merge medio
- 1 d 15 h
- PR fusionados (30 d)
- 8
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la 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.
Más de CommunityToolkit/AI
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
CommunityToolkit/AI#19 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
CommunityToolkit/AI#61 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
CommunityToolkit/AI#59 ·
-
Report all Cosmos DB emulator issuesQuizá libre de nuevo @adamsitnik la tomó hace 99 días y no hay ningún pull request abierto. Abierto
CommunityToolkit/AI#23 · 1 asignado ·
-
Make Azure SQL tests conditionalAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
CommunityToolkit/AI#20 ·
Todos los issues de CommunityToolkit/AI
Issues similares
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
ClickHouse/clickhouse-cs#689 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 7 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 65/100
vchelaru/FlatRedBall2#1481 ·
Los mantenedores suelen responder en 1 día
-
ai-chat bug trivial
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
area-signalr
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
dotnet/aspnetcore#69842 · 1 comentario ·
Los mantenedores suelen responder en 1 día