Azure / Azure/data-api-builder

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

Đang mở
#2,128 1 bình luận 0 reaction 0 người được giao Xem trên GitHub
refactor
Ngôn ngữ chính
C#
Star
1.5k
Fork
370
Merge trung bình
3 ngày 22 giờ
Pull request đã merge (30 ngày)
9

Mô tả

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_

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu bằng cách xem xét các model ParamDefinition và ColumnDefinition, cũng như cách DbType và SqlDbType được sử dụng trên các provider cơ sở dữ liệu. Xác định liệu DbType có còn cần thiết hay không và các field dành riêng cho provider nên được biểu diễn như thế nào mà không gây thêm gánh nặng cho các cơ sở dữ liệu không liên quan. Công việc được xem là hoàn tất khi thiết kế model đã được thống nhất và các phần sử dụng bị ảnh hưởng cùng các test phản ánh quyết định đó.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
csharp
Lĩnh vực
backend-api-design, databases
Loại issue
Tái cấu trúc
Độ khó
5/5
Thời gian dự kiến
Hơn một tuần
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Cần làm rõ
Mức phù hợp với người mới
20/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.