Skip to content
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

Account for very large text deltas (due to Copy & Paste etc.) #24

Open
adzialocha opened this issue Jan 17, 2025 · 0 comments
Open

Account for very large text deltas (due to Copy & Paste etc.) #24

adzialocha opened this issue Jan 17, 2025 · 0 comments
Labels
backend p2p networking, p2panda, etc. crdt Text-CRDT handling, data types, etc.

Comments

@adzialocha
Copy link
Member

adzialocha commented Jan 17, 2025

With gossip we have a max. message limit which is an issue, as these diffs can become very large. I believe that's due to a wrong use of Automerge but we'll run into the same issue sooner or later even with Automerge correctly used, for example when someone copy & pastes are large text into a document at once.

The current limit is 4096 bytes (temporarily increased to a higher number), that's roughly a text of 4096 characters, without any CRDT meta info or p2panda operation header.

This we can solve by chopping the text up into multiple, smaller "edits" and send one p2panda operation per edit

@adzialocha adzialocha added backend p2p networking, p2panda, etc. crdt Text-CRDT handling, data types, etc. labels Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend p2p networking, p2panda, etc. crdt Text-CRDT handling, data types, etc.
Projects
None yet
Development

No branches or pull requests

1 participant