Azure / Azure/data-api-builder

create a sub-classes of DbConnectionParam for MsSql Vs Other databases

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

Beschreibung

Might not be in this PR but something to consider: Can we create a sub-classes of DbConnectionParam for MsSql Vs Other databases (like `MsSqlDbConnectionParam`) and create parameter objects for sub-classes? Seems like for MsSql, just SqlDbType should be sufficient. Please open up an issue if this makes sense.

_Originally posted by @ayush3797 in https://github.com/Azure/data-api-builder/pull/2052#discussion_r1535456966_

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne damit, DbConnectionParam und den Code zu finden, der Parameterobjekte erstellt, und vergleiche anschließend, wie sich MsSql von anderen Datenbanken unterscheidet. Entscheide, ob separate Unterklassen wie MsSqlDbConnectionParam gerechtfertigt sind, und dokumentiere das vorgesehene Verhalten der Parameter. Als erledigt gilt die Aufgabe, wenn das Design vereinbart und der entsprechende Implementierungsumfang klar ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp
Bereich
backend-api-design, databases
Issue-Typ
Refactoring
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
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.