Azure / Azure/data-api-builder

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

オープン
#2,127 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
improvement mssql
主要言語
C#
スター
1.5k
フォーク
370
平均マージ
3日 22時間
マージ済み PR(30日)
9

説明

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_

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず DbConnectionParam とパラメーターオブジェクトを作成するコードを見つけ、MsSql が他のデータベースとどのように異なるかを比較します。MsSqlDbConnectionParam のような個別のサブクラスが必要かどうかを判断し、意図するパラメーターの動作を文書化します。設計について合意され、対応する実装範囲が明確になれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
csharp
領域
backend-api-design, databases
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。