Azure / Azure/data-api-builder
Assess refactor of code hydrating DbType for MsSql
- Lingua principale
- C#
- Stelle
- 1.5k
- Fork
- 370
- Merge medio
- 3g 22h
- PR unite (30g)
- 9
Descrizione
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:
https://github.com/Azure/data-api-builder/blob/0829d859301e9611c501b562cfc85e60e20b485a/src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs#L171
Need to assess this code duplication later and refactor accordingly if required.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Confronta i metodi di MsSqlMetadataProvider alle righe collegate in src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs con le relative definizioni nella classe base. Analizza PopulateColumnDefinitionWithHasDefaultAndDbType e FillSchemaForStoredProcedureAsync per determinare se l’hydration duplicata di DbType può essere sottoposta a refactoring senza modificare il comportamento. Il lavoro è concluso quando la valutazione è documentata tramite un refactoring appropriato o una decisione motivata di non effettuare il refactoring.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- csharp, sql
- Ambito
- databases
- Tipo di issue
- Refactoring
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Stato di attività
- Ferma
- Chiarezza
- Da chiarire
- Idoneità per principianti
- 25/100