Project

General

Profile

Actions

Task #3938

closed

Move top-level NLSR prefix registration

Added by Nicholas Gordon about 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Target version:
Start date:
01/30/2017
Due date:
% Done:

0%

Estimated time:

Description

Currently NLSR attempts to register its top-level prefix with the dispatcher before it has registered its localhost prefix. That should not be the case.


Related issues 2 (0 open2 closed)

Related to NLSR - Task #3930: Refactor NLSR to use dispatcher.ClosedNicholas Gordon01/20/2017

Actions
Blocked by NLSR - Feature #3728: Use ndn-cxx/mgmt/dispatcher for lsdb dataset publisherClosedLaqin Fan08/09/2016

Actions
Actions #1

Updated by Nicholas Gordon about 7 years ago

  • Status changed from New to Code review
Actions #2

Updated by Nicholas Gordon about 7 years ago

  • Related to Task #3930: Refactor NLSR to use dispatcher. added
Actions #3

Updated by Nicholas Gordon about 7 years ago

After reviewing the codebase and Ashlesh pointing something out, I see that it is not possible to cleanly replace the face-direct registration with the dispatcher in NLSR. This is because the dispatcher registration is asynchronous and does not provide a callback on top-level prefix registration success.
Currently the LSDBDatasetInterestHandler and PrefixUpdateProcessor set Face Interest filters directly. This needs to be done after the prefix registration, making it unsuitable to register the prefix with the dispatcher.

However, these two things can be refactored as prefix registrations with the dispatcher, which would allow the Face prefix registration itself to be moved into the dispatcher. The necessary refactoring would be to make these two modules accept some derived class of ControlParameters. This should not be too much work, because both of these modules at some point create a ControlParameters object out of the Interest they received.

I could have done it, but that's a little outside of the scope of this task.

Actions #4

Updated by Nicholas Gordon almost 7 years ago

  • Related to Feature #3728: Use ndn-cxx/mgmt/dispatcher for lsdb dataset publisher added
Actions #5

Updated by Nicholas Gordon almost 7 years ago

  • Related to deleted (Feature #3728: Use ndn-cxx/mgmt/dispatcher for lsdb dataset publisher)
Actions #6

Updated by Nicholas Gordon almost 7 years ago

  • Blocked by Feature #3728: Use ndn-cxx/mgmt/dispatcher for lsdb dataset publisher added
Actions #7

Updated by Nicholas Gordon over 6 years ago

  • Status changed from Code review to Closed
Actions

Also available in: Atom PDF