Summary on issues about PIT-CS - Githubissues

named-data / YaNFD

Yet another Named Data Networking Forwarding Daemon

https://pkg.go.dev/github.com/named-data/YaNFD

MIT License

12 stars 10 forks source link

Summary on issues about PIT-CS #42

Open zjkmxy opened 2 years ago

zjkmxy commented 2 years ago

Background: #38 #40

Performance is 25-50% worse after putting expiration in the same thread. Maybe the map deletion is slow. Possible bottleneck: https://github.com/named-data/YaNFD/blob/79b7e56d00715a9dc8f447ba68dfaec5dfc2f155/table/pit-cs-tree.go#L333
Till now PIT tokens are shown to be used in any tests. Not sure if it is YaNFD's problem.
pitTokenMap seems to be wrong: https://github.com/named-data/YaNFD/blob/79b7e56d00715a9dc8f447ba68dfaec5dfc2f155/table/pit-cs-tree.go#L25
csMap does not look good. It assumes the hashes have no conflict. https://github.com/named-data/YaNFD/blob/79b7e56d00715a9dc8f447ba68dfaec5dfc2f155/table/pit-cs-tree.go#L29 https://github.com/named-data/YaNFD/blob/79b7e56d00715a9dc8f447ba68dfaec5dfc2f155/table/pit-cs-tree.go#L372