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

Remove string table #45

Open
niloc132 opened this issue Mar 17, 2023 · 0 comments
Open

Remove string table #45

niloc132 opened this issue Mar 17, 2023 · 0 comments

Comments

@niloc132
Copy link
Member

This would be a breaking change to the wire format, and might not apply to all serialization streams.

Since we already track objects that have already been sent, we don't necessarily need a different mechanism for strings to be tracked in this way. Additionally, when sending as utf8, longs are sent as base64 strings in the main payload (up to 12 bytes) - the only effort being saved is in writing the length of the string before each occurrence.

This is a prerequisite to fully streaming reading of payloads, as otherwise no objects can be read until the full payload is received to resolve the type of the first object.

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

No branches or pull requests

1 participant