Project

General

Profile

Task #1832

Implement FaceTable class to monitor and track active faces

Added by Vince Lehman about 7 years ago. Updated about 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
RIB
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

The list of active faces should be maintained at a separate FaceTable class. The FaceTable class should:

• request Face dataset upon startup and periodically
    receive notifications using FaceMonitor #1744

• provide an API for RibManager to verify FaceId

• provide an EventEmitter for RibManager to delete Routes when a face is destroyed
#1

Updated by Junxiao Shi about 7 years ago

  • Target version set to v0.3
  • Start date deleted (08/08/2014)
#2

Updated by Junxiao Shi about 7 years ago

  • Blocks Bug #1852: fail to register prefix if multiple producers start together added
#3

Updated by Vince Lehman about 7 years ago

  • Description updated (diff)

Since the RibManager actually manages the RIB and currently applies FIB updates, wouldn't it be better to have the RibManager subscribe to the EventEmitter?

#4

Updated by Junxiao Shi about 7 years ago

It's wrong for RibManager to send any FIB update. FIB updates should be sent from a separate component.

See proposal and discussion at http://www.lists.cs.ucla.edu/mailman/private/nfd-dev/2014-June/000217.html

#5

Updated by Vince Lehman about 7 years ago

I agree that FIB updates should be handled in an additional component, but for this issue, if the RibManager maintains its same role, it should receive the face destroy event.

#6

Updated by Junxiao Shi about 7 years ago

In v0.2 we temporarily allowed RibManager to send FIB updates. This arrangement is not acceptable in v0.3.

#7

Updated by Vince Lehman about 7 years ago

So should I create a new task to implement a FibUpdater class? This issue does not deal with FIB updates.

#8

Updated by Alex Afanasyev about 7 years ago

  • Blocks deleted (Bug #1852: fail to register prefix if multiple producers start together)
#9

Updated by Vince Lehman about 7 years ago

  • Status changed from New to Closed

Also available in: Atom PDF