Azure / Azure/data-api-builder
Assess refactor of code hydrating DbType for MsSql
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- C#
- Estrellas
- 1.5k
- Forks
- 372
- Merge medio
- 3 d 22 h
- PR fusionados (30 d)
- 9
Descripción
The only additional step that MsSqlMetadataProvider.PopulateColumnDefinitionWithHasDefaultAndDbType performs different than its base class's definition is:
https://github.com/Azure/data-api-builder/blob/0829d859301e9611c501b562cfc85e60e20b485a/src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs#L109
and same for MsSqlMetadataProvider.FillSchemaForStoredProcedureAsync:
Need to assess this code duplication later and refactor accordingly if required.
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
Compara los métodos de MsSqlMetadataProvider en las líneas enlazadas de src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs con sus definiciones en la clase base. Sigue PopulateColumnDefinitionWithHasDefaultAndDbType y FillSchemaForStoredProcedureAsync para determinar si la carga duplicada de DbType puede refactorizarse sin cambiar el comportamiento. Se considera terminado cuando la evaluación queda documentada mediante un refactor adecuado o una decisión justificada de no refactorizar.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp, sql
- Área
- databases
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 25/100