Skip to content

Mergable persistance in Postgres tabular #246

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: beta
Choose a base branch
from

Conversation

jakubriedl
Copy link
Collaborator

Summary

Implements #237 to support merging updates from postgres to mergeable store.

How did you test this change?

WIP

WIP status

I'm still working through this feature and it needs more testing. But the existing code should outline intended solution.

@jakubriedl jakubriedl changed the base branch from main to beta May 13, 2025 11:49
@jamesgpearce jamesgpearce marked this pull request as ready for review May 15, 2025 18:48
@jamesgpearce
Copy link
Contributor

So I think we definitely need to crack this! I want to make sure we are capturing the full CRDT behavior though, not just row-level. CRDT hashes occur per value, per cell, per row, per table, and for all values and for all tables. We would need at least one 'tinybase' table to capture all of these.

Otherwise I would need to get very comfortable that row-level hashes only were good enough to resuscitate the data in a way that was robust enough to withstand the scrutiny of a full sync sequence.

(In any case, we also face the challenge of folks updating data through some other client and not having the CRDTs & hashes updated accordingly.

What do you think?

@jamesgpearce
Copy link
Contributor

(Do we want to keep these now you have your dedicated connector?)

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