Project

General

Profile

Actions

Task #4522

closed

Remove selectors from old interest format

Added by weijia yuan about 6 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Target version:
-
Start date:
02/28/2018
Due date:
% Done:

100%

Estimated time:

Description

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 2 (1 open1 closed)

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

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

Actions
Actions #1

Updated by weijia yuan about 6 years ago

  • Assignee changed from Alex Afanasyev to weijia yuan
Actions #2

Updated by Junxiao Shi about 6 years ago

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

Actions #3

Updated by Junxiao Shi about 6 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: https://github.com/3rd-ndn-hackathon/repo-sql/blob/47671612ef8b7c40931e5129c4d82626033f90a1/src/query-processor.cpp#L98-L99

Actions #4

Updated by Alex Afanasyev about 6 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 http://named-data.net/doc/NDN-packet-spec/current/name.html#canonical-order ?

Actions #6

Updated by Junxiao Shi over 5 years ago

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

Updated by weijia yuan over 5 years ago

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

Updated by Davide Pesavento about 5 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?

Actions #9

Updated by weijia yuan about 5 years ago

Yes this is finished and merged.

Actions #10

Updated by Junxiao Shi about 5 years ago

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

Also available in: Atom PDF