Task #4522

Remove selectors from old interest format

Added by weijia yuan almost 3 years ago. Updated about 2 years ago.

Target version:
Start date:
Due date:
% Done:


Estimated time:


Delete the code with selectors and related test, simplify the Interest-Data matching process.
The new version NDN interest simplify the selector function, so we need to update the corresponding part in repo-ng.

Related issues

Has duplicate ICE-AR Prototype - Task #4456: clean up the existing repo-ng codeClosedHaitao Zhang01/11/2018

Blocks repo-ng - Task #4523: Add Sync support for repoNewweijia yuan02/28/2018


Updated by weijia yuan almost 3 years ago

  • Assignee changed from Alex Afanasyev to weijia yuan

Updated by Junxiao Shi almost 3 years ago

This should wait for #4527, because repo-ng needs to handle CanBePrefix flag. Otherwise, it would be difficult to re-add it.


Updated by Junxiao Shi almost 3 years ago

Is there a specific reason that names are stored as blob type?

Yes. Names are binary, always. They should be processed and stored as such.

It's not easy to do name prefix matching using blob. If we store name as string in the database, then it will be easy to implement prefix matching.

In SQL there’s no “prefix matching”. Instead, you convert the query to “WHERE name >= query AND name < successor-of(query)”.
See example in repo-sql:


Updated by Alex Afanasyev almost 3 years ago

Junxiao, I think there is a problem with this logic. As far as I understand, it would work if we exclude T and L for the overall name in the stored value, but should have undefined results when comparing name as a whole. Am I misremembering our previous discussion of ?


Updated by Junxiao Shi over 2 years ago

  • Has duplicate Task #4456: clean up the existing repo-ng code added

Updated by weijia yuan over 2 years ago

  • Blocks Task #4523: Add Sync support for repo added

Updated by Davide Pesavento about 2 years ago

  • Subject changed from Remove selector from old interest format to Remove selectors from old interest format
  • Description updated (diff)
  • Status changed from New to In Progress

Is this done? if not, what's missing?


Updated by weijia yuan about 2 years ago

Yes this is finished and merged.


Updated by Junxiao Shi about 2 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF