Azure / Azure/data-api-builder
Assess refactor of code hydrating DbType for MsSql
- 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
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