Task #4832
closed
ndncatchunks: change version discovery logic
Added by Alex Afanasyev almost 6 years ago.
Updated over 5 years ago.
Description
Iterative discovery method should no longer be the default, as it is being removed quite soon (with removal of selectors). We should have changed to fixed or any other simple "discovery" long time ago.
- Start date deleted (
02/05/2019)
Version discovery based on metadata is being implemented in #4556. After that is done, as I suggested elsewhere, we can remove modular discovery altogether and use the user-provided name directly if it already contains a version component (equivalent to current 'fixed'), otherwise use metadata.
- Subject changed from Switch default discovery method to ndncatchunks: switch default discovery method
- Blocked by Task #4556: ndncatchunks: stop using Selectors added
- Assignee set to Chavoosh Ghasemi
- Subject changed from ndncatchunks: switch default discovery method to ndncatchunks: change version discovery logic
Change 5312 patchset 1 does not adequately implement #4832-1. It merely removes the -d option, but still have separate DiscoverVersionFixed and DiscoverVersionRealtime classes, i.e. version discovery modules.
To remove modular version discovery, combine all version discovery logic into one class or function that receives the user-supplied name as an input, and outputs a versioned name.
Furthermore, -d option cannot be abruptly removed because it breaks backwards compatibility of scripts that may still be using them. Instead, follow the standard deprecation procedure:
- Keep
-d option but don't include it in the help message or manpage.
- If
-d option is specified, ignore its value and print a warning message.
- Delete it after a standard deprecated cycle (90 days).
- Status changed from New to In Progress
- Status changed from In Progress to Code review
- % Done changed from 0 to 100
- Blocked by deleted (Task #4556: ndncatchunks: stop using Selectors)
- Blocks Task #4556: ndncatchunks: stop using Selectors added
- Status changed from Code review to Closed
- Related to Feature #5021: ndncatchunks: support unversioned data retrieval added
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by