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
Add sizes to fetch queue #1694
Add sizes to fetch queue #1694
Conversation
5a76900
to
20f3dde
Compare
/// The space this op belongs to | ||
pub space: KSpace, | ||
|
||
/// The source to fetch the op from | ||
pub source: FetchSource, | ||
|
||
/// The approximate size of the item | ||
pub size: Option<RoughInt>, | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just moved space and source into this struct (they were supplied independently before), but size
is the new addition here to the push parameters.
@@ -18,7 +18,7 @@ use holochain::{ | |||
use holochain_p2p::*; | |||
use holochain_sqlite::db::*; | |||
use kitsune_p2p::agent_store::AgentInfoSigned; | |||
use kitsune_p2p::gossip::sharded_gossip::test_utils::{check_ops_boom, create_agent_bloom}; | |||
use kitsune_p2p::gossip::sharded_gossip::test_utils::{check_ops_bloom, create_agent_bloom}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A'ight, looks good... I'll need to duplicate some of this on the publish side where I'm only pushing hashes will need to include the size as well.
Summary
Adds sizes to op hashes sent via historical gossip.
TODO: