Skip to content

feat: Add the FDv1 adapter synchronizer - #192

Draft
beekld wants to merge 1 commit into
bklimt/SDK-2865/fdv1-fallback-directivefrom
bklimt/SDK-2872/fdv1-adapter-synchronizer
Draft

feat: Add the FDv1 adapter synchronizer#192
beekld wants to merge 1 commit into
bklimt/SDK-2865/fdv1-fallback-directivefrom
bklimt/SDK-2872/fdv1-adapter-synchronizer

Conversation

@beekld

@beekldbeekld commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Gives the FDv1 fallback an implementation: an adapter that runs an existing FDv1 DataSource and turns its store writes into FDv2 change sets. A full store initialization becomes a full change set and an individual upsert becomes a partial one. A permanent FDv1 initialization failure surfaces to the orchestrator as a terminal error.

The fallback source is built once and re-subscribed on each activation, rather than rebuilt, because the FDv1 data source factory is not Send + Sync and cannot be captured in the synchronizer factory. Re-subscribing is safe, since each subscription starts a fresh stream or poll loop.

@beekld
beekldforce-pushed the bklimt/SDK-2872/fdv1-adapter-synchronizer branch from 2a693c3 to 97480dcCompareAugust 10, 2026 18:41
@beekld
beekldforce-pushed the bklimt/SDK-2872/fdv1-adapter-synchronizer branch from 97480dc to 960713dCompareAugust 10, 2026 18:46
@beekld
beekldforce-pushed the bklimt/SDK-2872/fdv1-adapter-synchronizer branch from 960713d to 94411c1CompareAugust 12, 2026 21:45
@beekld
beekldforce-pushed the bklimt/SDK-2872/fdv1-adapter-synchronizer branch from 94411c1 to b332740CompareAugust 12, 2026 22:10
@beekld
beekldforce-pushed the bklimt/SDK-2872/fdv1-adapter-synchronizer branch 2 times, most recently from d91dc97 to f862bdcCompareAugust 14, 2026 20:53
@beekld
beekldforce-pushed the bklimt/SDK-2872/fdv1-adapter-synchronizer branch 2 times, most recently from e2ce9ed to d1b0a6fCompareAugust 17, 2026 06:33
@beekld
beekldforce-pushed the bklimt/SDK-2872/fdv1-adapter-synchronizer branch from d1b0a6f to 6cc0802CompareAugust 20, 2026 00:35
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@beekld