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

idontwant with d=dlow-1 and iwant to pull messages! #969

Open
wants to merge 1 commit into
base: p2p-research
Choose a base branch
from

Conversation

ufarooqstatus
Copy link
Collaborator

Only for large messages. We relay large messages only to dlow-1 peers and send idontwant message to others. missing peers can use iwant to pull the large message

@arnetheduck
Copy link
Contributor

While this is an interesting idea:

  • why IDONTWANT? Why not IHAVE?
  • since sending the large messages takes time, it seems likely that peers will request it in most cases as they receive the notification before they receive the message - are there simulations looking into this effect?

@ufarooqstatus
Copy link
Collaborator Author

ufarooqstatus commented Mar 25, 2024

  • why IDONTWANT? Why not IHAVE?

IDontWant has two advantages over IHave:

  1. we need to maintain much smaller cache.
  2. On getting 1st iHave, the receiver generates IWant, although it is very likely that some other peer(s) are already sending the same message

@ufarooqstatus
Copy link
Collaborator Author

  • since sending the large messages takes time, it seems likely that peers will request it in most cases as they receive the notification before they receive the message - are there simulations looking into this effect?

Yes, and limited forwarding with/without IHaves showing better results for messages beyond 600KB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Experimental
Development

Successfully merging this pull request may close these issues.

2 participants