-
Notifications
You must be signed in to change notification settings - Fork 465
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: Add SQL server CDC guide #22541
Doc: Add SQL server CDC guide #22541
Conversation
b399869
to
bc72ff2
Compare
{{< tabs >}} | ||
{{< tab "Debezium 1.5+">}} | ||
|
||
1. Create a connector configuration file and save it as `register-sqlserver.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.
What about using the link to the configuration example that includes also additional details rather than the 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.
That example in the Debezium docs is missing some crucial configuration settings like the Avro Converter and the CSR details so we can't use it directly.
* `common-config` | ||
* `common-utils` | ||
|
||
1. Create a connector configuration file and save it as `register-sqlserver.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.
Same as sqlserver.json
. Maybe I'm missing something.
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.
Hm I am not 100% sure what you mean here?
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.
It was the same as the other comment, but probably you have the same answer about crucial config missing! So nothing to do here.
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.
Ah I see! Thank you for the clarification.
GROUP BY field1; | ||
``` | ||
|
||
## Known Limitations |
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.
Also Microsoft has a Known issues and errors with CDC
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.
Good finding! I've added a link to this in the list
Co-authored-by: Joaquin Colacci <[email protected]>
Motivation
Adding a SQL Server CDC guide that follows the exact same structure as the existing MySQL guide.
Also added a demo here for this.
Closes MaterializeInc/database-issues#3220