fix: Update DataSystemConfig to accept list of synchronizers #404
+200
−169
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.
Note
Medium Risk
Touches core data-source failover logic in
FDv2and changes a public/experimental config shape, which could affect initialization and fallback behavior across environments.Overview
Updates
DataSystemConfigandConfigBuilderto replace theprimary_synchronizer/secondary_synchronizermodel with an ordered list ofsynchronizers, validating at least one synchronizer when set and updating contract-test config parsing accordingly.Refactors
FDv2synchronizer execution to iterate/fallback across N synchronizers (including removal of permanently failing sources, index-based recovery to the first synchronizer, and an explicit directive to switch to the FDv1 fallback synchronizer), and updates/extends unit tests to cover the new API and behaviors. CI contract tests are also bumped to v3v3.0.0-alpha.3and YAML quoting is normalized.Written by Cursor Bugbot for commit 9e2c297. This will update automatically on new commits. Configure here.