-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[hotfix][docs] Correct reference link for DB2 docs #1683
Conversation
2294146
to
054a56f
Compare
Thanks @Jiabao-Sun for this contribution. I left a litter comments. On line 163 of the file and "latest-offset". Please see Startup Reading Positionsection A space is required here besides, LGTM |
f97ace2
to
9519246
Compare
Thanks @RookieGao for the detailed review. |
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.
Thanks for your work, @Jiabao-Sun. Just a little suggestion.
docs/content/connectors/db2-cdc.md
Outdated
@@ -246,8 +246,7 @@ Data Type Mapping | |||
<table class="colwidths-auto docutils"> | |||
<thead> | |||
<tr> | |||
<th class="text-left" style="width:30%;">Db2 type<a href="https://dev.mysql.com/doc/man/8.0/en/data-types. | |||
html"></a></th> | |||
<th class="text-left" style="width:30%;"><a href="https://www.ibm.com/docs/en/db2-for-zos/12?topic=elements-data-types">Db2 type</a></th> |
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.
I think this is better to link to https://www.ibm.com/docs/en/db2/11.5?topic=elements-data-types.
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.
Thanks @ruanhang1993 for this suggestion.
Fixed.
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. Thanks.
Correct reference link for DB2 docs