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

Change NFD's CS replacement policy to LRU #5

Merged
merged 1 commit into from
Nov 11, 2024
Merged

Change NFD's CS replacement policy to LRU #5

merged 1 commit into from
Nov 11, 2024

Conversation

Pesa
Copy link
Member

@Pesa Pesa commented Oct 25, 2024

The default was changed over 6 years ago (named-data/NFD@32e7609). I guess it never trickled down to the testbed config.

See also: https://redmine.named-data.net/issues/4728

The default was changed over 6 years ago
named-data/NFD@32e7609
@Pesa Pesa requested review from pulsejet and tianyuan129 October 25, 2024 23:13
@@ -61,13 +56,13 @@ face_system

udp
{
listen yes
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just for consistency with the other config sections that have this explicitly set (even though it's the default).

port {{udp_port}}
enable_v4 yes
enable_v6 yes

idle_timeout 600
unicast_mtu 1452
keep_alive_interval 25
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nonfunctional, it was never implemented in NFD. The config file parser still accepts it for some reason but I'd like to drop it at some point.

@Pesa
Copy link
Member Author

Pesa commented Nov 11, 2024

Any comments?

@pulsejet
Copy link
Contributor

Thanks 🙏

@pulsejet pulsejet merged commit c10131d into main Nov 11, 2024
2 checks passed
@pulsejet pulsejet deleted the cs-lru branch November 11, 2024 19:17
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.

2 participants