-
Notifications
You must be signed in to change notification settings - Fork 2k
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
[DOC ] Update data type mapping #1802
Conversation
@wangxiaojing Thanks for your contributions! Can you revoke unchanged lines? And you can also correct type mapping in |
@@ -1,11 +1,13 @@ | |||
# SQLServer CDC Connector | |||
|
|||
The SQLServer CDC connector allows for reading snapshot data and incremental data from SQLServer database. This document describes how to setup the SQLServer CDC connector to run SQL queries against SQLServer databases. | |||
The SQLServer CDC connector allows for reading snapshot data and incremental data from SQLServer database. This document |
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.
Do not format files.
@@ -369,7 +405,7 @@ Data Type Mapping | |||
</tr> | |||
<tr> | |||
<td>tinyint</td> | |||
<td>TINYINT</td> | |||
<td>SMALLINT</td> |
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.
Please fix corresponding all types test
.
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.
LGTM
* [DOC ] Update data type mapping * Delete doc format,change all types testcase * Delete format * code format * Update sink table schema Co-authored-by: wangxiaojing <[email protected]>
SQL Server 's type tinyint with values from 0 to 255.
Flink 's type tinyint with values from -128 to 127.
2.float Mapping to double