Azure / Azure/data-api-builder
Check if DbType is required, since now we are introducing SqlDbType
オープン
refactor
- 主要言語
- C#
- スター
- 1.5k
- フォーク
- 370
- 平均マージ
- 3日 22時間
- マージ済み PR(30日)
- 9
説明
Bloating a ParamDefinition/ColumnDefinition with fields not relevant for every database seems not to be the best approach here.
_Originally posted by @ayush3797 in https://github.com/Azure/data-api-builder/pull/2052#discussion_r1535448998_
コントリビューションガイド
調査の方向性
まず、ParamDefinition モデルと ColumnDefinition モデル、および各データベースプロバイダーで DbType と SqlDbType がどのように使用されているかを確認します。DbType が引き続き必要かどうか、また関係のないデータベースに負担をかけずにプロバイダー固有のフィールドをどのように表現すべきかを判断します。モデル設計について合意され、影響を受ける使用箇所とテストがその判断を反映していれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- csharp
- 領域
- backend-api-design, databases
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 20/100