sql-json-schema-mapper maps decimal to integer
See original GitHub issueHello, I was trying to write tests for my PR and saw this.
function mapSQLTypeToOpenAPIType (sqlType) { // TODO support more types /* istanbul ignore next */ switch (sqlType) { case 'int': return 'integer' case 'integer': return 'integer' case 'tinyint': return 'integer' case 'smallint': return 'integer' case 'decimal': return 'integer'
Is it designed like this or is it a bug?
Issue Analytics
- State:
- Created 9 months ago
- Comments:8 (8 by maintainers)
Top Results From Across the Web
Numeric types — Understanding JSON Schema 2020-12 ...
There are two numeric types in JSON Schema: integer and number. ... Therefore, the presence or absence of a decimal point is not...
Read more >Schema and data type mapping in copy activity - Microsoft Learn
This article describes how the Azure Data Factory copy activity perform schema mapping and data type mapping from source data to sink data....
Read more >Why would you use a string in JSON to represent a decimal ...
The main reason to transfer numeric values in JSON as strings is to eliminate any loss of precision or ambiguity in transfer. It's...
Read more >CData JDBC Driver for JSON - Data Type Mapping
Schema.NET JDBC SQLite
int, integer, int32 Int32 int integer
smallint, short, int16 Int16 short integer
double, float, real Double double double
Read more >MySQL / MSSQL : numeric.mapping doesn't work for ... - GitHub
I can't get numeric.mapping to work with MySQL and Confluent Platform ... transactions ( txn_id INT, customer_id INT, amount DECIMAL(5,2), ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
we are expecting a number @cesarvspr
Please take a look when possible https://github.com/platformatic/platformatic/pull/506