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

feat(clustering): entity streaming based cp-dp protocol #11630

Closed

Conversation

bungle
Copy link
Member

@bungle bungle commented Sep 21, 2023

Summary

Uses streaming protocol for CP-DP reconfigure to avoid huge blob inflation/deflation + json encoding/decoding.

A more complete enhancement can be found in PR #11632.

KAG-2673
KAG-2670

@bungle bungle added pr/do not merge pr/discussion This PR is being debated. Probably just a few details. labels Sep 21, 2023
@bungle bungle force-pushed the feat/cp-dp-entity-streaming-without-lmdb-streaming branch from ff013b0 to eb6807d Compare September 25, 2023 12:06
@bungle bungle force-pushed the feat/cp-dp-entity-streaming-without-lmdb-streaming branch 5 times, most recently from a34bd09 to 87074b6 Compare September 25, 2023 12:42
@bungle bungle force-pushed the feat/cp-dp-entity-streaming-without-lmdb-streaming branch 11 times, most recently from deeacdd to efcfa3d Compare September 29, 2023 18:56
@bungle bungle force-pushed the feat/cp-dp-entity-streaming-without-lmdb-streaming branch 9 times, most recently from d7cdc2d to 08036b3 Compare October 5, 2023 10:55
bungle added 2 commits October 5, 2023 23:43
### Summary

Uses streaming protocol for CP-DP reconfigure to avoid huge blob
inflation/deflation + json encoding/decoding.

Signed-off-by: Aapo Talvensaari <[email protected]>
@bungle bungle force-pushed the feat/cp-dp-entity-streaming-without-lmdb-streaming branch from 08036b3 to a324bf4 Compare October 5, 2023 20:44
@kikito
Copy link
Member

kikito commented Oct 10, 2023

Closing as this is already contained in #11632

@kikito kikito closed this Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core/clustering core/db pr/discussion This PR is being debated. Probably just a few details. size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants