Azure / Azure/data-api-builder

Check if DbType is required, since now we are introducing SqlDbType

オープン
#2,128 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
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

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

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