Skip to content

Use exported datastream merge functions, skip partial on cache miss#65

Merged
dontlaugh merged 2 commits into
mainfrom
diff-algo
Mar 17, 2026
Merged

Use exported datastream merge functions, skip partial on cache miss#65
dontlaugh merged 2 commits into
mainfrom
diff-algo

Conversation

@dontlaugh

@dontlaugh dontlaugh commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

Re-use PartialCompany/PartialUser from schematic-go/datastream

On cache miss for partial messages, log a warning
and skip rather than writing incomplete data. Remove dead pre-merge
unmarshal code. Update go.mod to schematic-go@f539b4e (main HEAD).

Re-use PartialCompany/PartialUser from schematic-go/datastream

On cache miss for partial messages, log a warning
and skip rather than writing incomplete data. Remove dead pre-merge
unmarshal code. Update go.mod to schematic-go@f539b4e (main HEAD).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dontlaugh dontlaugh changed the title Import diffing functions from schematic-go Use exported datastream merge functions, skip partial on cache miss Mar 16, 2026
@dontlaugh
dontlaugh marked this pull request as ready for review March 16, 2026 18:03
@dontlaugh
dontlaugh requested a review from a team as a code owner March 16, 2026 18:03
Comment thread go.mod
github.com/schematichq/rulesengine v0.1.14
github.com/schematichq/schematic-datastream-ws v0.2.7
github.com/schematichq/schematic-go v1.4.3
github.com/schematichq/schematic-go v1.4.4-0.20260316174403-f539b4e4a607

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An unreleased version we can use if we want.

@dontlaugh
dontlaugh merged commit e028eee into main Mar 17, 2026
3 checks passed
@dontlaugh
dontlaugh deleted the diff-algo branch March 17, 2026 19:07
Sign up for free to 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.

2 participants