RDSC-5610 Multi-source documentation fixes - #4030
Merged
andy-stark-redis merged 1 commit intoSep 17, 2026
Merged
Conversation
ZdravkoDonev-redis
approved these changes
Sep 17, 2026
2 tasks
6 tasks
EliShteinman
added a commit
to EliShteinman/docs
that referenced
this pull request
Sep 19, 2026
Trims the RDI classic-to-Flink migration and upgrade guides, replacing the 2.0.0 upgrade section with a single warning about the new Flink default. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
See https://redislabs.atlassian.net/browse/RDSC-5610
Fixes on top of #4029 that restore some of the latest changes to the docs that got lost during the conflict resolution in that PR.
Note
Low Risk
Documentation-only edits to installation guides; no runtime or API behavior changes.
Overview
Restores and tightens RDI classic→Flink migration and upgrade docs after conflict resolution (RDSC-5610).
In migration-classic-to-flink.md, Flink is described as the default processor; drain verification uses
redis-di describe(notdescribe default) with a shorter Pending explanation; the Redis Insight drain-check subsection and several operational notes (e.g. avoidingredis-di stop, change-log retention detail, cross-checks withXLEN/XPENDING, and “don’t truncate streams” guidance) are removed. Rollback wording is simplified to “setprocessors.typetoclassic”.In upgrade.md, the duplicate early “Upgrading to RDI 2.0.0” processor-default section is removed; the Flink-default behavior for 2.0.0 is called out in a
warningunder Enabling the Flink processor (unsetprocessors.type→ Flink on next deploy; setclassicbefore upgrade to stay on classic).Reviewed by Cursor Bugbot for commit b062bd2. Bugbot is set up for automated code reviews on this repo. Configure here.