-
Notifications
You must be signed in to change notification settings - Fork 307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(warehouse): added additional datatypes for warehouse integrations #2971
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #2971 +/- ##
==========================================
+ Coverage 53.51% 53.59% +0.08%
==========================================
Files 350 350
Lines 54452 54452
==========================================
+ Hits 29139 29184 +45
+ Misses 23649 23612 -37
+ Partials 1664 1656 -8
... and 5 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
…t.integration-additional-datatypes
c8f241e
to
b289470
Compare
…t.integration-additional-datatypes
DeepSource issues are resolved in this #2964 |
…t.integration-additional-datatypes
…t.integration-additional-datatypes
…t.integration-additional-datatypes
This PR is considered to be stale. It has been open 20 days with no further activity thus it is going to be closed in 5 days. To avoid such a case please consider removing the stale label manually or add a comment to the PR. |
"bigint": "int", | ||
"int": "int", | ||
"smallint": "int", | ||
"tinyint": "int", | ||
"integer": "int", | ||
|
||
"numeric": "float", | ||
"decimal": "float", | ||
"money": "float", | ||
"small money": "float", | ||
"real": "float", | ||
"float": "float", | ||
"double precision": "float", | ||
|
||
"char": "string", | ||
"nchar": "string", | ||
"varchar": "string", | ||
"nvarchar": "string", | ||
"text": "string", | ||
"ntext": "string", | ||
"binary": "string", | ||
"image": "string", | ||
"varbinary": "string", | ||
"uniqueidentifier": "string", | ||
|
||
"date": "datetime", | ||
"datetimeoffset": "datetime", | ||
"datetime2": "datetime", | ||
"smalldatetime": "datetime", | ||
"datetime": "datetime", | ||
"time": "datetime", | ||
"timestamp with time zone": "datetime", | ||
"timestamp": "datetime", | ||
"jsonb": "json", | ||
"bit": "boolean", | ||
|
||
"bit": "boolean", | ||
|
||
"jsonb": "json", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These datatypes we have added:
int, money, small money, uniqueidentifier
Description
warehouse_rudder_missing_datatype
stats generated during the fetching of the schema from the warehouse.Corresponding documentation links:
Notion Ticket
https://www.notion.so/rudderstacks/Additional-datatypes-for-warehouse-Integrations-a765dda055db40589f0b2eedddce56ad?pvs=4
Security