Repository navigation
RDSC-6142: Document external RDI collectors for proof of concept projects - #4204
Open
ZdravkoDonev-redis wants to merge 6 commits into
Open
ZdravkoDonev-redis wants to merge 6 commits into
ZdravkoDonev-redis wants to merge 6 commits into
Conversation
Contributor
Contributor
🧠 Redis MemoryFound 8 related items from repository history:
Memory updated at 15d02a2 |
ZdravkoDonev-redis
marked this pull request as ready for review
October 12, 2026 12:24
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
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.
Document external collectors for RDI proof of concept projects
Readers can currently select
type: externalwithout an input contract or operating guidance. This change adds Use an external collector under Data pipelines → Prepare source databases, with navigation weight 2. It explains the recommended PoC scope, production downsides, integration options, stream/event contract, configuration, transformation job, and a complete Python writer. It also explains the differences in source controls and visibility and links the guide from current source guidance, existing mentions, and relevant CLI references.The recommendation does not declare external collectors unsupported. The same pipeline configuration and deployment command apply to self-managed VM and Kubernetes installations. Cloud setup and the 1.19.1 snapshot are unchanged.
Version requirement: The guide now requires self-managed RDI 2.2.0 or later, which includes the connectionless external-source deployment fix. Earlier versions reject connectionless external sources during deployment validation. The validation evidence below records the versions actually tested; it does not claim a new RDI 2.2.0 qualification.
Validation
redis/rdi-flink-processor:2.0.0image passed read, create, update, and delete operations, transformation/key routing, the resulting hash after a repeated create, malformed JSON and missing key/value rejection to dead-letter queues, schema/payload wrappers withschema: null, and discovery/routing of another source stream. These are processor data-path checks, not a complete installation/deployment qualification or general delivery-guarantee tests.validate_configservice path with the documented job, both with and without table validation. Disabling table validation does not bypass the issue.redis/rdi-api:2.0.0service functions reject per-source external start, stop, and reset with HTTP 400, and metadata discovery with HTTP 423. Kubernetes access was mocked; these are service checks, not an end-to-end CLI/control-plane qualification. Whole-pipeline stop/reset remained accepted at that layer.git diff --checkpasses.Limits and review notes
jupyter-example) outside this change. The focused RDI build passes.externalwithx-internal-values: ["external"]. This change leaves the marker, generated schema visibility, and runtime behavior unchanged. Review this deliberate public guidance against that marker before publication.Parallel qualification evidence
Reviewed RDSC-6142 and RDI PR #2875 at
928f605ac3a0647087f94f7bbfad8e1cf665d84f. Its Helm/Flink/single-source restrictions are smoke-runner constraints, not new product restrictions. The ticket and PR report a fresh-machine Helm installation with Redis TLS/mTLS, connectionless deployment, no managed source collector, exact target checks for all four operations, and actual whole-pipeline stop/start with queued delivery. That qualification used a development build with the API fix applied, not the exact PR commit or the released 2.0.0 images. Exact-commit smoke CI was pending at review time; the implementation PR is open. This is reference evidence and was not rerun in this docs task.The public guide now explains manual VM configuration after installation, separates writer credentials from target secrets, and creates the pipeline before setting target secrets on installations without a default pipeline. It does not link private implementation evidence from the public pages. Full VM qualification remains unconfirmed.
Note
Low Risk
Documentation-only changes to RDI integration content; no application code, APIs, or runtime behavior modified.
Overview
Adds a new Use an external collector guide under Prepare source databases, framing
type: externalas a proof-of-concept path (not recommended for production) and documenting the Debezium-compatible Redis stream contract, PoC setup (config.yaml, jobs, deploy/secrets flow), a sample PythonXADDwriter, verification steps, and how operations differ from RDI-managed collectors.Existing RDI pages and CLI references are updated to link to the guide and clarify behavior: PoC callouts on overview/architecture/pipeline entry points, target-only secrets and deploy ordering for external sources, expanded
externalsource config notes, per-sourcestart/stop/resetlimitations (including--source), noredis-di scaffoldtemplate, and release-note pointers to the new doc.Reviewed by Cursor Bugbot for commit 15d02a2. Bugbot is set up for automated code reviews on this repo. Configure here.