Task #4642
closed
Update producers across platform to unregister prefixes instead of shutting down face
Added by Eric Newberry over 6 years ago.
Updated over 6 years ago.
Description
Per #4636, to ensure that the last transmitted Data is not lost, producers need to unregister prefixes to cause Face::processEvents() to return, instead of calling Face::shutdown(). These changes need to be made to producers across the entire platform.
- Related to Task #4636: Modify docs to show producers unregistering prefixes to shut down faces added
- Status changed from New to In Progress
- % Done changed from 0 to 10
It appears that the following producers already utilize the unregisterPrefix() mechanism to halt their faces (or potentially some other mechanism that doesn't utilize shutdown() right after sending a Data packet):
- ndnpingserver
- ndnputchunks
- ndn-pib
- name-based-access-control producers
- repo-ng
- ChronoShare
- ndncert
- ndn-autoreg
- ndn-autoconfig-server
Meanwhile, the follow producers utilize the shutdown() mechanism and need to modified to utilize it (if they're still in use):
- ndnpoke
- ndn-traffic-server
- ndn-atmos cxx-producer
I removed ndnputchunks from the list of producers needing to be modified, since it only calls Face::shutdown() in response to a prefix registration error.
- Status changed from In Progress to Code review
- % Done changed from 10 to 70
Patches for ndnpoke and ndn-traffic-server have been pushed to Gerrit for review.
Note that I will not be changing instances where Face::shutdown() is called in response to a SIGINT, as exiting immediately seems likely to be the intended behavior in response to that signal.
- % Done changed from 70 to 100
Susmit told me not to worry about cxx-producer in ndn-atmos, as ndn-atmos is unable to compile against current versions of ndn-cxx (because it still uses C++11).
The fix for ndn-traffic-server has been merged.
- Status changed from Code review to Closed
The fix for ndnpoke has been merged.
- Status changed from Closed to Feedback
Reopening. Eric reports that the fix for ndn-traffic-server is not working as expected.
- Status changed from Feedback to Code review
- Status changed from Code review to Closed
Also available in: Atom
PDF
Updated by 
Updated by