data-api-builder icon indicating copy to clipboard operation
data-api-builder copied to clipboard
verified publisher icon Azure

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

Open abhishekkumams opened this issue 1 year ago • 1 comments 

          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

abhishekkumams avatar Mar 22 '24 12:03 abhishekkumams

Need more context for this issue. Which database use the param/column definition types? SqlDbType is only applicable to MSSQL and not pg/mysql/cosmos.

seantleonard avatar Mar 22 '24 16:03 seantleonard