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

Spread TIE summary over multiple TIDE packets #92

Open
brunorijsman opened this issue Jul 8, 2020 · 0 comments
Open

Spread TIE summary over multiple TIDE packets #92

brunorijsman opened this issue Jul 8, 2020 · 0 comments
Labels
enhancement New feature or request

Comments

@brunorijsman
Copy link
Owner

Currently every RIFT-Python node always summarizes its entire TIE database in a single TIDE packet.

If the TIE database is very large (contains many TIEs) then the TIDE packet can get very large.

The one and only TIDE packet is always sent in a single UDP packet. If the UDP packet is larger than the interface MTU, UDP fragmentation will split it into multiple IP packets which the receiver UDP will reassemble.

Consider enhancing RIFT-Python to spread the TIE database summary over multiple TIDE packets. The RIFT TIDE packet has a range field for exactly this purpose.

This potentially can avoid UDP fragmentation, which could potentially be a good thing.

The downside is that it brings in a lot of extra complexity, and at this point it is not clear to me that the extra complexity is worth it (since RIFT-Python cannot support huge networks in the first place for completely other reasons, or maybe for multiple reasons including this one).

@brunorijsman brunorijsman added the enhancement New feature or request label Jul 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant