Azure / Azure/data-api-builder

Assess refactor of code hydrating DbType for MsSql

Aperta
#1,914 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
refactor
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

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.