-
Notifications
You must be signed in to change notification settings - Fork 2
docs: add 5.1 migration guide #53
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
Conversation
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 it would be nice to have a top-level Migration
section, under which we have two separate pages, one for Confluent Platform
and another for Confluent Cloud
. We can place the Migration
section just before Reference
?
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
@nvitucci would you please take a look at this one? |
@ali-ince @dhrudevalia I have a few questions first:
|
Hey @nvitucci,
@dhrudevalia do you mind making the changes @nvitucci suggested above? |
Thank you, @ali-ince. So:
I'll add a couple more specific suggestions to each page as well, but these were the main decision points. Thanks! |
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've left several (mostly minor) comments to improve readability.
Co-authored-by: Nicola Vitucci <nicola.vitucci@gmail.com>
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.
Few more minor comments. Other than that, it's good for me.
modules/ROOT/pages/migration/5.1/migration-confluent-cloud.adoc
Outdated
Show resolved
Hide resolved
modules/ROOT/pages/migration/5.1/migration-confluent-cloud.adoc
Outdated
Show resolved
Hide resolved
Co-authored-by: Nicola Vitucci <nicola.vitucci@gmail.com>
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
Issue
From 5.0.x -> 5.1 many configuration keys have been updated and changed. To make the upgrade process easier the 5.0.6 connector prints an automatically migrated configuration to the log
Docs update