Azure / Azure/data-api-builder

Assess refactor of code hydrating DbType for MsSql

Offen
#1,914 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
refactor
Vorherrschende Sprache
C#
Sterne
1.5k
Forks
370
Ø Merge
3 T. 22 Std.
Gemergte PRs (30 T.)
9

Beschreibung

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.

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Vergleiche die MsSqlMetadataProvider-Methoden an den verlinkten Zeilen in src/Core/Services/MetadataProviders/MsSqlMetadataProvider.cs mit ihren Definitionen in der Basisklasse. Verfolge PopulateColumnDefinitionWithHasDefaultAndDbType und FillSchemaForStoredProcedureAsync, um festzustellen, ob die duplizierte DbType-Hydratisierung refaktoriert werden kann, ohne das Verhalten zu ändern. Als abgeschlossen gilt die Aufgabe, wenn die Bewertung durch einen geeigneten Refactor oder eine begründete Entscheidung gegen einen Refactor dokumentiert ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp, sql
Bereich
databases
Issue-Typ
Refactoring
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.