Skip to content

Improving SQL Server setup documentation when using custom types - #3998

Open
ilianiliev-redis wants to merge 3 commits into
mainfrom
improving-sqlserver-setup-guide
Open

ilianiliev-redis wants to merge 3 commits into
mainfrom
improving-sqlserver-setup-guide

Conversation

@ilianiliev-redis

@ilianiliev-redis ilianiliev-redis commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

Note

Low Risk
Documentation-only changes with no runtime or security code impact.

Overview
Adds section 5 to the SQL Server RDI prep guide for tables that use user-defined types, documenting that the Debezium user needs GRANT REFERENCES ON TYPE::<schema>.<type-name> and showing the misleading primary-key error you can see without that grant.

Also fixes the missing newline at end of file on the schema-change note at the bottom of the doc.

Reviewed by Cursor Bugbot for commit 43fa592. Bugbot is set up for automated code reviews on this repo. Configure here.

@github-actions

github-actions Bot commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

🧠 Redis Memory

Found 7 related items from repository history:

Memory updated at 43fa592

@github-actions

Copy link
Copy Markdown
Contributor

@andy-stark-redis andy-stark-redis left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple of minor style suggestions but otherwise LGTM.

Comment thread content/integrate/redis-data-integration/data-pipelines/prepare-dbs/sql-server.md Outdated
Comment thread content/integrate/redis-data-integration/data-pipelines/prepare-dbs/sql-server.md Outdated
ilianiliev-redis and others added 2 commits September 16, 2026 17:35
…e-dbs/sql-server.md

Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
…e-dbs/sql-server.md

Co-authored-by: andy-stark-redis <164213578+andy-stark-redis@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants