Skip to content

peer, main: allow for queuing multiple invvects #91

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

Conversation

kcalvinalvin
Copy link
Contributor

For utreexo tx inv messages, the invvects will be related to each other as the invvect for the tx will be matched with the position invs. It'll look like so:

[InvTypeUtreexoTx][TXID1]
[InvTypeProofHash][packed positions1.1] (4 positions) [InvTypeProofHash][packed positions1.2] (1 position)

At the moment the invvects can only be queued one at a time which may result in the InvTypeProofHash to be separated from the InvTypeUtreexoTx. Allowing for queuing multiple invvects eliminates this problem.

For utreexo tx inv messages, the invvects will be related to each other
as the invvect for the tx will be matched with the position invs. It'll
look like so:

[InvTypeUtreexoTx][TXID1]
[InvTypeProofHash][packed positions1.1] (4 positions)
[InvTypeProofHash][packed positions1.2] (1 position)

At the moment the invvects can only be queued one at a time which may
result in the InvTypeProofHash to be separated from the
InvTypeUtreexoTx. Allowing for queuing multiple invvects eliminates this
problem.
@kcalvinalvin kcalvinalvin force-pushed the 2023-12-19-add-ability-to-queue-multiple-invs branch from 0e1b6d9 to 4480a0e Compare December 21, 2023 06:44
@kcalvinalvin kcalvinalvin merged commit 56725df into utreexo:main Dec 21, 2023
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

Successfully merging this pull request may close these issues.

1 participant